Skip to content

Commit

Permalink
bump python linting from 3.7->3.8 against linting failure
Browse files Browse the repository at this point in the history
  • Loading branch information
tuncK committed Oct 6, 2023
1 parent d582d8b commit 1127085
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 11 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/lint.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ jobs:
runs-on: ubuntu-latest
strategy:
matrix:
python-version: ['3.7', '3.12']
python-version: ['3.8', '3.12']
env:
LINT_PATH: 'lib/galaxy/dependencies/pinned-lint-requirements.txt'
TYPE_PATH: 'lib/galaxy/dependencies/pinned-typecheck-requirements.txt'
Expand Down
11 changes: 4 additions & 7 deletions lib/galaxy/dependencies/pinned-lint-requirements.txt
Original file line number Diff line number Diff line change
@@ -1,10 +1,7 @@
attrs==23.1.0
flake8==5.0.4
flake8-bugbear==23.3.12
importlib-metadata==4.2.0
flake8==6.1.0
flake8-bugbear==23.9.16
mccabe==0.7.0
pycodestyle==2.9.1
pyflakes==2.5.0
pycodestyle==2.11.0
pyflakes==3.1.0
ruff==0.0.292
typing_extensions==4.7.1
zipp==3.15.0
6 changes: 3 additions & 3 deletions lib/galaxy/dependencies/update_lint_requirements.sh
Original file line number Diff line number Diff line change
Expand Up @@ -4,14 +4,14 @@ set -e

# This script updates the pinned requirements for linting.
# The lint requirements are split from the the other ones due to incompatible
# dependencies: flake8 requires importlib-metadata 4.2, while the Galaxy's core
# dependencies depend on importlib-metadata 4.13.
# dependencies.
# Flake8 v6.0.0 supports python v3.8.1+ only.

THIS_DIRECTORY="$(cd "$(dirname "$0")" > /dev/null && pwd)"

update_pinned_reqs() {
VENV=$(mktemp -d "${TMPDIR:-/tmp}/$1_venv.XXXXXXXXXX")
python3.7 -m venv "${VENV}"
python3.8 -m venv "${VENV}"
. "${VENV}/bin/activate"
pip install --upgrade pip setuptools
pip install -r "${THIS_DIRECTORY}/$1-requirements.txt"
Expand Down

0 comments on commit 1127085

Please sign in to comment.