Skip to content

[red-knot] Understand other legacy typing aliases#14998

Merged
AlexWaygood merged 9 commits intoastral-sh:mainfrom
InSyncWithFoo:rk-typing-aliases
Dec 17, 2024
Merged

[red-knot] Understand other legacy typing aliases#14998
AlexWaygood merged 9 commits intoastral-sh:mainfrom
InSyncWithFoo:rk-typing-aliases

Conversation

@InSyncWithFoo
Copy link
Contributor

Summary

Resolves #14997.

Test Plan

Markdown tests.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 16, 2024

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

@MichaReiser MichaReiser added the ty Multi-file analysis & type inference label Dec 16, 2024
Copy link
Member

@AlexWaygood AlexWaygood left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. I think we'll probably have to rewrite a lot of this when we add support for generics, but I suppose there's no harm in improving our understanding of these annotations as best we can in the meantime

Copy link
Member

@AlexWaygood AlexWaygood left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@AlexWaygood AlexWaygood merged commit 7c2e7cf into astral-sh:main Dec 17, 2024
@InSyncWithFoo InSyncWithFoo deleted the rk-typing-aliases branch December 17, 2024 09:38
dcreager added a commit that referenced this pull request Dec 17, 2024
* main:
  [red-knot] Explicitly test diagnostics are emitted for unresolvable submodule imports (#15035)
  Fix stale File status in tests (#15030)
  [red-knot] Basic support for other legacy `typing` aliases (#14998)
  feat(AIR302): extend the following rules (#15015)
  [`perflint`] Simplify finding the loop target in `PERF401` (#15025)
  [red-knot] Avoid undeclared path when raising conflicting declarations (#14958)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ty Multi-file analysis & type inference

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[red-knot] Understand other legacy typing aliases

3 participants