Skip to content

bpo-41515: prevent get_type_hints from raising KeyError#21816

Closed
efahl wants to merge 2 commits into
python:masterfrom
efahl:master
Closed

bpo-41515: prevent get_type_hints from raising KeyError#21816
efahl wants to merge 2 commits into
python:masterfrom
efahl:master

Conversation

@efahl

@efahl efahl commented Aug 10, 2020

Copy link
Copy Markdown
Contributor

efahl added 2 commits August 10, 2020 12:26
typing.get_type_hints raises KeyError on synthetic modules that don't appear in sys.modules.
@the-knights-who-say-ni

Copy link
Copy Markdown

Hello, and thanks for your contribution!

I'm a bot set up to make sure that the project can legally accept this contribution by verifying everyone involved has signed the PSF contributor agreement (CLA).

Recognized GitHub username

We couldn't find a bugs.python.org (b.p.o) account corresponding to the following GitHub usernames:

@efahl

This might be simply due to a missing "GitHub Name" entry in one's b.p.o account settings. This is necessary for legal reasons before we can look at this contribution. Please follow the steps outlined in the CPython devguide to rectify this issue.

You can check yourself to see if the CLA has been received.

Thanks again for the contribution, we look forward to reviewing it!

@gvanrossum gvanrossum left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Sorry for sitting on this so long. Can you prepare a short blurb too?

@gvanrossum

Copy link
Copy Markdown
Member

Also please rebase, it might fix the macOS test failure.

@tirkarthi

Copy link
Copy Markdown
Member

I have created #25352 with attribution to author since I don't have permission to update the PR. Thanks.

@gvanrossum

Copy link
Copy Markdown
Member

Closed in favor of GH-25352.

@gvanrossum gvanrossum closed this Apr 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants