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

bpo-41824: Add versionadded for typing.ForwardRef docs #24224

Merged
merged 2 commits into from
Feb 9, 2021

Conversation

Fidget-Spinner
Copy link
Member

@Fidget-Spinner Fidget-Spinner commented Jan 15, 2021

typing.ForwardRef officially became part of the documented public API in GH-14216 (19 Jun 2019). I'm interpolating a little and guessing it landed in Python 3.7.4 according to PEP 537's schedule https://www.python.org/dev/peps/pep-0537/#id3.

https://bugs.python.org/issue41824

@Fidget-Spinner
Copy link
Member Author

I'm guessing this should be backported to 3.9 and 3.8 (along with skip news)?

@gvanrossum gvanrossum merged commit da21f7b into python:master Feb 9, 2021
@bedevere-bot
Copy link

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

@Fidget-Spinner
Copy link
Member Author

@gvanrossum Thanks Guido! Should this be backported to 3.9/3.8 too?

@gvanrossum
Copy link
Member

@gvanrossum Thanks Guido! Should this be backported to 3.9/3.8 too?

Sure.

@Fidget-Spinner
Copy link
Member Author

@gvanrossum Thanks Guido! Should this be backported to 3.9/3.8 too?

Sure.

Sorry to trouble, can you add the labels for them please? I don't have permissions to. Though I'm not sure if the bot will pick it up if the label is applied after the PR is merged rather than before. Either way I'm fine with manually backporting them too.

@miss-islington
Copy link
Contributor

Thanks @Fidget-Spinner for the PR, and @gvanrossum for merging it 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖 I'm not a witch! I'm not a witch!

@miss-islington
Copy link
Contributor

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

@gvanrossum
Copy link
Member

Looks like the bot picked it up!

@miss-islington
Copy link
Contributor

Sorry, @Fidget-Spinner and @gvanrossum, I could not cleanly backport this to 3.8 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker da21f7b6e1fd5bd3e78931a06c5eb694f6335233 3.8

@miss-islington
Copy link
Contributor

Sorry @Fidget-Spinner and @gvanrossum, I had trouble checking out the 3.9 backport branch.
Please backport using cherry_picker on command line.
cherry_picker da21f7b6e1fd5bd3e78931a06c5eb694f6335233 3.9

@Fidget-Spinner
Copy link
Member Author

Fidget-Spinner commented Feb 9, 2021

Aww I guess I should've done it manually from the start then. Thanks for the help anyways!

@bedevere-bot bedevere-bot removed the needs backport to 3.9 only security fixes label Feb 9, 2021
@bedevere-bot
Copy link

GH-24493 is a backport of this pull request to the 3.9 branch.

Fidget-Spinner added a commit to Fidget-Spinner/cpython that referenced this pull request Feb 9, 2021
@bedevere-bot
Copy link

GH-24494 is a backport of this pull request to the 3.8 branch.

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 issue skip news
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants