[ty] Avoid double-analyzing tuple in Final subscript by charliermarsh · Pull Request #21828 · astral-sh/ruff (original) (raw)
Avoid double-analyzing tuple in [ty] Avoid double-analyzing tuple in Final subscriptFinal subscript
- Fix
ty#1793links to be proper markdown hyperlinks - Add extended tests showing type qualifiers still work despite invalid type arguments:
- ClassVar: show it's still a ClassVar (invalid-attribute-access on instance assignment)
- Final: show it's still Final (invalid-assignment on reassignment)
- InitVar: show it's still an InitVar (init signature, unresolved-attribute on access)
- Add test case for ty#1768 (ClassVar[int,] trailing comma syntax)
🤖 Generated with Claude Code
Co-Authored-By: Claude Opus 4.5 noreply@anthropic.com
dcreager added a commit that referenced this pull request
- origin/main:
[ty] Add test case for fixed panic (#21832)
[ty] Avoid double-analyzing tuple in
Finalsubscript (#21828) [flake8-bandit] Fix false positive when using non-standardCSafeLoaderpath (S506). (#21830) Add minimal-size build profile (#21826)
dcreager added a commit that referenced this pull request
- origin/main:
[ty] Add test case for fixed panic (#21832)
[ty] Avoid double-analyzing tuple in
Finalsubscript (#21828) [flake8-bandit] Fix false positive when using non-standardCSafeLoaderpath (S506). (#21830) Add minimal-size build profile (#21826) [ty] Allowtuple[Any, ...]to assign totuple[int, *tuple[int, ...]](#21803) [ty] Support renaming import aliases (#21792) [ty] Add redeclaration LSP tests (#21812) [ty] more detailed description of "Size limit on unions of literals" in mdtest (#21804) [ty] Complete support forParamSpec(#21445) [ty] Update benchmark dependencies (#21815)
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
[ Show hidden characters]({{ revealButtonHref }})