Enforce modern annotations syntax #4368
Merged
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 of changes
Relates to #2345 (comment) , as more type hints and annotations are added. Using a modern annotations syntax is preferable, reduces extraneous typing imports and allows more type subscriptions.
I've merged the
ignore
andextend-ignore
ignore entries in Ruff becauseextend-ignore
is obsolete https://docs.astral.sh/ruff/settings/#lint_extend-ignoreBumped ruff in pre-commit to latest.
Bumped pytest-ruff to latest (https://github.com/businho/pytest-ruff/releases/tag/v0.3.2 should have a much cleaner error output now)
Edit: I keep forgetting this comes from the skeleton, I opened jaraco/skeleton#126
Pull Request Checklist
newsfragments/
. (no user-facing changes)(See documentation for details)