Skip to content

[3.9] bpo-16781: In 'exec' doc, add 'nonlocal' to 'yield' and 'return' (GH-2446) #24470

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

Merged
merged 1 commit into from
Feb 7, 2021

Conversation

miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Feb 7, 2021

These 3 statements cannot be used at module scope -- nor in exec with one namespace.
(cherry picked from commit 0ec57e2)

Co-authored-by: Terry Jan Reedy tjreedy@udel.edu

https://bugs.python.org/issue16781

…honGH-2446)

These 3 statements cannot be used at module scope -- nor in exec with one namespace.
(cherry picked from commit 0ec57e2)

Co-authored-by: Terry Jan Reedy <tjreedy@udel.edu>
@miss-islington
Copy link
Contributor Author

@dzenanz: Status check is done, and it's a success ✅ .

@bedevere-bot bedevere-bot added docs Documentation in the Doc dir awaiting review labels Feb 7, 2021
@terryjreedy terryjreedy merged commit 863eb71 into python:3.9 Feb 7, 2021
@miss-islington miss-islington deleted the backport-0ec57e2-3.9 branch February 7, 2021 14:14
@dzenanz
Copy link

dzenanz commented Feb 8, 2021

I am really confused by this mention. Any explanation as to why? I think I proposed a patch to Python years ago, but it was unrelated to this.

@terryjreedy
Copy link
Member

A backport PR identical to the original PR based on a tracker issue is not the place to ask questions. Please try python-list, and be more clear about what you are asking, and omit unrelated stuff.

@dzenanz
Copy link

dzenanz commented Feb 8, 2021

@miss-islington called me out in a comment in this PR. Why?

@terryjreedy
Copy link
Member

terryjreedy commented Feb 8, 2021

Sorry, without seeing the bogus @dzenanz mention above, which should have been @terryjreedy , I did not understand your question. I am puzzled too.
@Mariatta (miss-islington bot author) Random glitch? Dzenan has no relation to this project.

@dzenanz
Copy link

dzenanz commented Feb 8, 2021

It also happened in 3.8 PR, but not in the original one.

@terryjreedy
Copy link
Member

Ah. I figured it out. When I merged #24469, I copied the updated current title to replace the original title which github annoying puts in the merge box instead of the current title. When selecting, I accidentally omitted the 9 at the end. PR 2446 is your PR. Sorry.

@dzenanz
Copy link

dzenanz commented Feb 8, 2021

I 🤣. Mystery resolved!

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.

5 participants