Skip to content

Commit

Permalink
Merge pull request #57 from laracroft37/ci-linting
Browse files Browse the repository at this point in the history
Add more CI linting based on black, pycodestyle, and pydocstyle.

This pull request extends the linting workflow with more checks and
adapts the code base for these to pass.
  • Loading branch information
pevogam authored Sep 2, 2024
2 parents 017cd33 + a1f568f commit fed13d2
Show file tree
Hide file tree
Showing 19 changed files with 3,259 additions and 1,556 deletions.
10 changes: 7 additions & 3 deletions .github/workflows/lint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ name: Lint Check
on: [push, pull_request]

jobs:
mypy:
lint:
runs-on: ubuntu-latest

steps:
Expand All @@ -17,8 +17,12 @@ jobs:
- name: Install dependencies
run: |
python -m pip install --upgrade pip
pip install mypy
pip install mypy black pycodestyle pydocstyle
- name: Run mypy
- name: Run linters
run: |
mypy guibot
black --check --diff --color guibot
# only excluded checks are conflicts with black and within pycodestyle
pycodestyle --ignore=E203,E501,W503 guibot
pydocstyle guibot
2 changes: 1 addition & 1 deletion guibot/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -14,10 +14,10 @@
# along with guibot. If not, see <http://www.gnu.org/licenses/>.

"""
Package with the complete guibot modules and functionality.
SUMMARY
------------------------------------------------------
Package with the complete guibot modules and functionality.
INTERFACE
Expand Down
266 changes: 195 additions & 71 deletions guibot/calibrator.py

Large diffs are not rendered by default.

Loading

0 comments on commit fed13d2

Please sign in to comment.