[3.9] bpo-16781: In 'exec' doc, add 'nonlocal' to 'yield' and 'return' (GH-2446) by miss-islington · Pull Request #24470 · python/cpython (original) (raw)

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 andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Conversation8 Commits1 Checks0 Files changed

Conversation

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

[ Show hidden characters]({{ revealButtonHref }})

miss-islington

@terryjreedy @miss-islington

…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

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

terryjreedy

@dzenanz

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

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

@terryjreedy

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

@terryjreedy

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