FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

bpo-41824: Add versionadded for typing.ForwardRef docs by Fidget-Spinner · Pull Request #24224 · python/cpython · GitHub

/ cpython Public

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

Merged
gvanrossum merged 2 commits into
python:masterfrom
Fidget-Spinner:patch-1
Feb 9, 2021
Merged

bpo-41824: Add versionadded for typing.ForwardRef docs#24224
gvanrossum merged 2 commits into
python:masterfrom
Fidget-Spinner:patch-1

Conversation

Fidget-Spinner commented Jan 15, 2021
edited
Loading

Copy link
Copy Markdown
Member

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

Copy link
Copy Markdown
Member Author

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

gvanrossum merged commit da21f7b into python:master Feb 9, 2021

Copy link
Copy Markdown

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

Copy link
Copy Markdown
Member Author

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

Copy link
Copy Markdown
Member

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

Sure.

Copy link
Copy Markdown
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.

Copy link
Copy Markdown
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!

Copy link
Copy Markdown
Contributor

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

Copy link
Copy Markdown
Member

Looks like the bot picked it up!

Copy link
Copy Markdown
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

Copy link
Copy Markdown
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 commented Feb 9, 2021
edited
Loading

Copy link
Copy Markdown
Member Author

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

Copy link
Copy Markdown

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

Copy link
Copy Markdown

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

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL