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

gh-48496: Added example and link to faq for UnboundLocalError in reference #93068

Merged
merged 4 commits into from Dec 22, 2022

Conversation

slateny
Copy link
Contributor

@slateny slateny commented May 22, 2022

#48496

https://docs.python.org/3/faq/programming.html#why-am-i-getting-an-unboundlocalerror-when-the-variable-has-a-value

https://docs.python.org/3/reference/executionmodel.html#resolution-of-names

A comment on the issue mentioned putting an example in the tutorial, but skipped it since it's already got an FAQ entry, and that the scopes tutorial makes no mention of NameError/UnboundLocalError.

@bedevere-bot bedevere-bot added docs Documentation in the Doc dir awaiting review labels May 22, 2022
@netlify
Copy link

netlify bot commented Dec 10, 2022

Deploy Preview for python-cpython-preview ready!

Name Link
🔨 Latest commit 7026c3a
🔍 Latest deploy log https://app.netlify.com/sites/python-cpython-preview/deploys/6394088f35567c0009d22065
😎 Deploy Preview https://deploy-preview-93068--python-cpython-preview.netlify.app/reference/executionmodel
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@kumaraditya303 kumaraditya303 requested a review from CAM-Gerlach Dec 21, 2022
Copy link
Member

@CAM-Gerlach CAM-Gerlach left a comment

One comment, otherwise generally LGTM from me

Doc/reference/executionmodel.rst Outdated Show resolved Hide resolved
Co-authored-by: C.A.M. Gerlach <CAM.Gerlach@Gerlach.CAM>
Copy link
Member

@CAM-Gerlach CAM-Gerlach left a comment

LGTM from me, thanks

Copy link
Contributor

@hauntsaninja hauntsaninja left a comment

Thank you!

cc @JelleZijlstra

@hauntsaninja hauntsaninja merged commit f3db68e into python:main Dec 22, 2022
15 checks passed
@miss-islington
Copy link
Contributor

miss-islington commented Dec 22, 2022

Thanks @slateny for the PR, and @hauntsaninja for merging it 🌮🎉.. I'm working now to backport this PR to: 3.10, 3.11.
🐍🍒🤖

@bedevere-bot
Copy link

bedevere-bot commented Dec 22, 2022

GH-100442 is a backport of this pull request to the 3.11 branch.

@bedevere-bot
Copy link

bedevere-bot commented Dec 22, 2022

GH-100443 is a backport of this pull request to the 3.10 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 22, 2022
…n reference (pythonGH-93068)

(cherry picked from commit f3db68e)

Co-authored-by: Stanley <46876382+slateny@users.noreply.github.com>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 22, 2022
…n reference (pythonGH-93068)

(cherry picked from commit f3db68e)

Co-authored-by: Stanley <46876382+slateny@users.noreply.github.com>
miss-islington added a commit that referenced this pull request Dec 22, 2022
…rence (GH-93068)

(cherry picked from commit f3db68e)

Co-authored-by: Stanley <46876382+slateny@users.noreply.github.com>
miss-islington added a commit that referenced this pull request Dec 22, 2022
…rence (GH-93068)

(cherry picked from commit f3db68e)

Co-authored-by: Stanley <46876382+slateny@users.noreply.github.com>
@slateny slateny deleted the s/48496 branch Dec 23, 2022
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.

None yet

6 participants