MNT: upgrade ruff pre-commit hook, move its configuration out of pre-commit-config.yaml and fix lints#371
Conversation
31f6bdd to
7038f67
Compare
|
Seeing a couple failures with which indicate that the python interpreter used in CI is older than 3.10. Footnotes
|
7038f67 to
8f4785d
Compare
|
updated base branch to develop |
|
python updated to 3.14 from sources on the self-hosted runner. Should work fine. |
8f4785d to
59cb74e
Compare
|
thanks. Pre-commit still failed on CI but it looks like some unrelated networking error. |
59cb74e to
fd38d2f
Compare
|
linting passed, but now the env under which tests are run looks broken. |
|
tests are failing because somehow #367 is not included upstream. Can you rebase ? |
|
I don't think that's it. I did check that commits from #367 are in my branch's history. In any case rebasing locally turned into a no-op |
0fb065c to
60dd0e9
Compare
Moving ruff's config to
ruff.tomlallows running ruff's LSP in an IDE and see lints beforegit commitis invoked.I also upgraded the tool to a much more recent version and applied newly discovered lints.