Skip to content

Commit

Permalink
Fix broken linter workflows (#8933)
Browse files Browse the repository at this point in the history
* bandit 1.8.1 removed some of the checks, so remove them from our
  config.

* pylint 2.x doesn't support Python 3.12, and GitHub just switched their
  `ubuntu-latest` runners to Ubuntu 24.04, where 3.12 is the default. It's
  complicated to update pylint, so revert the runner back to Ubuntu 22.04
  for now.
  • Loading branch information
SpecLad authored Jan 13, 2025
1 parent af8bfe3 commit 42bc345
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
4 changes: 1 addition & 3 deletions .bandit
Original file line number Diff line number Diff line change
@@ -1,9 +1,7 @@
[bandit]
# B101 : assert_used
# B102 : exec_used
# B320 : xml_bad_etree
# B404 : import_subprocess
# B406 : import_xml_sax
# B410 : import_lxml
skips: B101,B102,B320,B404,B406,B410
skips: B101,B102,B404,B406
exclude: **/tests/**,tests
2 changes: 1 addition & 1 deletion .github/workflows/pylint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ name: Pylint
on: pull_request
jobs:
Linter:
runs-on: ubuntu-latest
runs-on: ubuntu-22.04 # TODO: update pylint to 3.x and change this back to ubuntu-latest
steps:
- uses: actions/checkout@v4
- id: files
Expand Down

0 comments on commit 42bc345

Please sign in to comment.