Skip to content

Commit

Permalink
Merge pull request #1130 from teoparvanov/flake8-linter-fixes
Browse files Browse the repository at this point in the history
Flake8 linter fixes + updating PR lint rules
  • Loading branch information
teoparvanov authored Nov 11, 2024
2 parents e87627f + 73ce1d3 commit 522208a
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 7 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/lint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -27,5 +27,5 @@ jobs:
run: |
python -m pip install --upgrade pip
pip install -r linters-requirements.txt
- name: Lint using built-in script
run: bash shell/lint.sh
- name: Lint with OpenFL-specific rules
run: bash shell/lint.sh
8 changes: 3 additions & 5 deletions .github/workflows/ubuntu.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,14 +16,12 @@ jobs:
uses: actions/setup-python@v3
with:
python-version: "3.8"
- name: Install dependencies
- name: Install linters
run: |
python -m pip install --upgrade pip
pip install -r linters-requirements.txt
pip install .
- name: Lint with flake8
run: |
flake8 --show-source
- name: Lint with OpenFL-specific rules
run: bash shell/lint.sh

pytest-coverage: # from pytest_coverage.yml
needs: lint
Expand Down

0 comments on commit 522208a

Please sign in to comment.