Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(commands): support 3.12's type statement and TypeAliasType #1128

Merged
merged 13 commits into from
Dec 8, 2023

Conversation

shiftinv
Copy link
Member

Summary

(most of the recent typing-related commits were a prerequisite for this; #1122 is the one remaining blocker for this, afaict)

This change adds support for type annotations declared using the type statement from PEP 695; the deferred evaluation is really quite neat for users, but it made evaluating signatures a bit more tricky, especially when you throw forwardrefs into the mix.

Checklist

  • If code changes were made, then they have been tested
    • I have updated the documentation to reflect the changes
    • I have formatted the code properly by running pdm lint
    • I have type-checked the code by running pdm pyright
  • This PR fixes an issue
  • This PR adds something new (e.g. new method or parameters)
  • This PR is a breaking change (e.g. methods or parameters removed/renamed)
  • This PR is not a code change (e.g. documentation, README, ...)

Reason for this is that GenericAlias proxies its `__origin__`'s attributes;
if we have a GenericAlias with a TypeAliasType `__origin__`,
`hasattr(tp, "__value__")` will match even though we're really looking at a GenericAlias.
@shiftinv shiftinv added t: enhancement New feature s: blocked Issue/PR is blocked by other issues s: needs review Issue/PR is awaiting reviews labels Oct 26, 2023
@shiftinv shiftinv added this to the disnake v2.10 milestone Oct 26, 2023
@shiftinv shiftinv removed the s: blocked Issue/PR is blocked by other issues label Nov 17, 2023
@shiftinv shiftinv merged commit 85cf393 into master Dec 8, 2023
27 checks passed
@shiftinv shiftinv deleted the feature/py-3.12-typealiastype branch December 8, 2023 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
s: needs review Issue/PR is awaiting reviews t: enhancement New feature
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant