Skip to content

Commit

Permalink
Add pre-commit to linting stage
Browse files Browse the repository at this point in the history
  • Loading branch information
Jason committed Jul 6, 2023
1 parent d3e7286 commit 0a43556
Showing 1 changed file with 3 additions and 19 deletions.
22 changes: 3 additions & 19 deletions .github/workflows/testing.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,27 +25,11 @@ jobs:

- name: Install dependencies
run: |
python -m pip install --upgrade pip
pip install -r requirements.txt
pip install -r requirements-testing.txt
- name: Lint with pycodestyle
run: |
pip install pycodestyle
pycodestyle src/maggma
- name: Lint with mypy
run: |
pip install mypy mypy-boto3
mypy src/maggma
pip install pre-commit
- name: Lint with flake8
- name: Run pre-commit
run: |
pip install flake8
# stop the build if there are Python syntax errors or undefined names
flake8 --count --show-source --statistics src/maggma
# exit-zero treats all errors as warnings.
flake8 --count --exit-zero --max-complexity=20 --statistics src/maggma
pre-commit run --all-files --show-diff-on-failure
test:
needs: lint
Expand Down

0 comments on commit 0a43556

Please sign in to comment.