[pre-commit.ci] pre-commit autoupdate - #833
Merged
Merged
Conversation
updates: - [github.com/astral-sh/ruff-pre-commit: v0.15.22 → v0.16.0](astral-sh/ruff-pre-commit@v0.15.22...v0.16.0)
for more information, see https://pre-commit.ci
Ruff 0.16.0 enables 413 rules by default (up from 59), so lint.extend-select started pulling in hundreds of never-selected rules (128 FA100 errors alone). Switch to an explicit lint.select with the same list — it already contains E and F in full, which covered the old defaults — to preserve the previous rule set. Also rename the hook id from the legacy alias "ruff" to "ruff-check". Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Ruff 0.16.0 enables 413 rules by default (up from 59). Keep lint.extend-select so the new defaults apply, and fix the resulting 140 errors instead of opting out: - FA100: add `from __future__ import annotations` everywhere and simplify Optional[X]/quoted annotations accordingly (auto-fix), then drop the typing imports left unused (the fix is not offered in __init__.py files) - PYI036: use proper `__exit__` annotations in tools/utils.py - TRY004: raise TypeError instead of ValueError in Location.__lt__ - PERF402, C400, C414, PIE810, PYI034: auto-fixed - DTZ005: noqa, the summary timestamps intentionally stay naive local time so the output format does not change - BLE001: noqa, any mypy crash must become a fatal-build-error The unquoted Queue annotation in tools/mypy needs a pylint pragma: multiprocessing.Queue is a factory function at runtime, so pylint rightly calls it unsubscriptable, while type checkers see the class. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
The PERF402 fix replaced the append loop with a conditional extend, which leaves mypy unable to infer the list's element type. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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.
updates: