Skip to content

Conversation

@mhils
Copy link
Contributor

@mhils mhils commented Aug 20, 2021

As agreed on in bpo-44926, this PR adds a small note to typing.get_type_hints explaining that it fails on type aliases with forward references imported from other modules.

I've tried to keep the text short. The first commit has a slightly longer version if we want more details/explanation here, but I think this should be enough. :)

https://bugs.python.org/issue44926

Copy link
Member

@Fidget-Spinner Fidget-Spinner left a comment

Choose a reason for hiding this comment

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

Looks good, I just have one comment.

mhils and others added 2 commits August 20, 2021 14:52
Co-authored-by: Ken Jin <28750310+Fidget-Spinner@users.noreply.github.com>
Copy link
Member

@Fidget-Spinner Fidget-Spinner left a comment

Choose a reason for hiding this comment

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

Great. Thanks!

@miss-islington
Copy link
Contributor

Thanks @mhils for the PR, and @gvanrossum for merging it 🌮🎉.. I'm working now to backport this PR to: 3.10.
🐍🍒⛏🤖

@bedevere-bot
Copy link

@gvanrossum: Please replace # with GH- in the commit message next time. Thanks!

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Aug 20, 2021
… refs (pythonGH-27859)

(cherry picked from commit 16b9be4)

Co-authored-by: Maximilian Hils <git@maximilianhils.com>
@bedevere-bot bedevere-bot removed the needs backport to 3.10 only security fixes label Aug 20, 2021
@bedevere-bot
Copy link

GH-27860 is a backport of this pull request to the 3.10 branch.

@mhils mhils deleted the get-type-hints-note branch August 20, 2021 14:38
ambv pushed a commit that referenced this pull request Aug 22, 2021
… refs (GH-27859) (GH-27860)

(cherry picked from commit 16b9be4)

Co-authored-by: Maximilian Hils <git@maximilianhils.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation in the Doc dir skip news

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants