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

Use __future__ annotations #265

Closed
wants to merge 59 commits into from
Closed

Use __future__ annotations #265

wants to merge 59 commits into from

Conversation

MikeMitusov
Copy link
Contributor

@MikeMitusov MikeMitusov commented Jan 20, 2024

I've added: from __future__ import annotations to all important files

@MikeMitusov MikeMitusov changed the title [fix-future] Use __future__ annotations Use __future__ annotations Jan 20, 2024
@MikeMitusov MikeMitusov deleted the fix-future branch January 20, 2024 15:37
@MikeMitusov MikeMitusov restored the fix-future branch January 20, 2024 15:37
@MikeMitusov MikeMitusov reopened this Jan 20, 2024
@MikeMitusov MikeMitusov deleted the fix-future branch January 20, 2024 18:14
@johnbywater
Copy link
Collaborator

What's wrong @MikeMitusov? Just let me resolve the trouble with singledispatch and future annotations?

johnbywater added a commit that referenced this pull request Jan 20, 2024
…tions

Fix future annotations (from #265) plus some changes to singledispatchmethod.
@johnbywater
Copy link
Collaborator

I merged all these changes in #266. Thank you very much for addressing this important aspect of the code. I spent a couple of hours figuring out what was broken with the singledispatchmethod code and fixed that (it was trying to resolve annotations too early and also resolving annotations required classmethod to be set with a __wrapped__ attribute so that the typing module can get the globals. But it all seems to work now. I appreciate your contribution, and hope you don't mind that I folded these things into this PR (you had closed your original one and removed the branch so I didn't know what else to do).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants