Repository navigation
Type Parameter Modernization: Fixes, Latest Version - #287
Draft
FrancescoPezzella wants to merge 25 commits into
Draft
FrancescoPezzella wants to merge 25 commits into
FrancescoPezzella wants to merge 25 commits into
Conversation
FrancescoPezzella
changed the base branch from
main
to
typing-recipes-2-cst-structure
October 13, 2026 14:21
FrancescoPezzella
added this pull request to stack #277
October 13, 2026 14:21
Collaborator
Author
|
mark most bugs with TODO: there are 0 instances found in 10+ projects. they are also often the result of syntax that goes against conventions, tool should be working more testing will be doen against more projects and results will be reported |
FrancescoPezzella
removed this pull request from stack #277
October 16, 2026 11:58
FrancescoPezzella
added this pull request to stack #296
October 16, 2026 12:15
FrancescoPezzella
changed the title
FrancescoPezzella
removed this pull request from stack #296
October 16, 2026 13:51
FrancescoPezzella
changed the base branch from
typing-recipes-2-cst-structure
to
main
October 16, 2026 13:52
FrancescoPezzella
changed the base branch from
main
to
typing-recipes-2-cst-structure
October 16, 2026 15:07
…unction using a TypeVar that can't be converted is now left whole and reported unsafe.
…used only by a nested function now gets its [T] there instead of on the enclosing function.
…uff I001 in the CLI Needed imports are added once per file after the last import, and the CLI merges and sorts them with ruff.
Keeps `# noqa` and similar comments on the declaration; comment lines above it are tracked under "Need to fix".
…them and drop its trailing comment on removal Fixes `from <origin> import Any` for names the origin imports, and the stray `# noqa` left by orphan removal.
FrancescoPezzella
force-pushed
the
typing-recipes-2-further-fixes
branch
from
October 16, 2026 15:25
e409e7b to
5f14bb4
Compare
FrancescoPezzella
added this pull request to stack #295
October 16, 2026 15:25
FrancescoPezzella
changed the title
… a fix. Document that rewritten files are written with LF line endings as intended behaviour instead of listing it under Need to fix.
…ports as TypeVarCheck non-goals.
…al behaviour and one example each.
…igration report. Expose each declaration's kind from TypeVarCheck so the CLI can count them.
…unctions were converted. Simplify the migration report: plain-language descriptions, one shared documentation link, and fewer, end-to-end CLI tests.
…nctions use them, since variance applies to generic classes only.
…nused declaration removed in the migration report.
…level and explain its two causes.
FrancescoPezzella
force-pushed
the
typing-recipes-2-further-fixes
branch
from
October 19, 2026 16:17
f71b9c7 to
799de1a
Compare
…upported version for the tool is 3.12.
…les, and report them with an Ignored column.
…ocks, not only Python version checks.
…nverted, removed or localized per name, and the files left for manual review by reason.
… untouched and keeps their imports in other files. Document that the tool only sees the scanned project, so unused type parameters in a shared module are removed unless it is passed to --ignore.
…instead of copying their declarations into the importing file.
… style its headings, paths and names as Markdown. Drop the redundant ruff import-cleanup sentence.
…ause of imports later localized, merging both passes into one report.
This branch has not been deployed
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.
No description provided.