🔧 Update linting rules and pre-commit config (py310 floor, drop setup-cfg-fmt, bump revs) - #564
Merged
MiWeiss merged 5 commits intoJun 12, 2026
Merged
🔧 Update linting rules and pre-commit config (py310 floor, drop setup-cfg-fmt, bump revs)#564MiWeiss merged 5 commits into
MiWeiss merged 5 commits into
Conversation
Copy link
Copy Markdown
Collaborator
Sorry, something went wrong.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Mechanical changes from the bumped hooks, mainly pyupgrade --py310-plus (PEP 585/604 annotations), removal of now-unused typing imports, and black 26 / isort 8 reformatting. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
MiWeiss
force-pushed
the
546-pre-commit-config
branch
from
22339d0 to
19e1e97
Compare
June 12, 2026 11:25
The previous local pre-commit runs spuriously skipped tests/middleware_tests/test_latex_encoding.py, which made the pre-commit CI job fail on the sweep commit. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
# Conflicts: # bibtexparser/entrypoint.py # bibtexparser/library.py # bibtexparser/middlewares/latex_encoding.py # bibtexparser/middlewares/names.py
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixes the pre-commit config drift:
Includes the hook sweep after all
This was originally intended config-only, deferring the --all-files rewrite to a later PR to avoid conflicts with #560/#562/#563. That plan turned out not to be viable: the pre-commit CI job runs the updated hooks on all files, so the config-only head was red (see checks on 81e580d). The second commit therefore applies the hooks to the codebase: PEP 585/604 typing modernization, removal of then-unused typing imports, black 26 / isort 8 reformatting. Hooks were run to a fixpoint (they need several passes, as pyupgrade runs after autoflake/isort); the full test suite passes.
Merge-order note: the sweep touches 25 files mechanically and will conflict with #560/#562/#563. Suggested order: merge those first, then rebase this branch and re-run pre-commit run --all-files to refresh the sweep. The annotation changes here may also partly subsume #563 / the 538-type-annotations work.
The optional ruff consolidation from the issue is left out as a separate decision.
Fixes #546
🤖 Generated with Claude Code