Skip to content

[3.8] bpo-38469: Handle named expression scope with global/nonlocal keywords (GH-16755) - #16760

Merged
ambv merged 1 commit into
python:3.8from
miss-islington:backport-fd5c414-3.8
Oct 14, 2019
Merged

[3.8] bpo-38469: Handle named expression scope with global/nonlocal keywords (GH-16755)#16760
ambv merged 1 commit into
python:3.8from
miss-islington:backport-fd5c414-3.8

Conversation

@miss-islington

@miss-islington miss-islington commented Oct 14, 2019

Copy link
Copy Markdown
Contributor

The symbol table handing of PEP572's assignment expressions is not resolving correctly the scope of some variables in presence of global/nonlocal keywords in conjunction with comprehensions.
(cherry picked from commit fd5c414)

Co-authored-by: Pablo Galindo Pablogsal@gmail.com

https://bugs.python.org/issue38469

pythonGH-16755)

The symbol table handing of PEP572's assignment expressions is not resolving correctly the scope of some variables in presence of global/nonlocal keywords in conjunction with comprehensions.
(cherry picked from commit fd5c414)

Co-authored-by: Pablo Galindo <Pablogsal@gmail.com>
@pablogsal

pablogsal commented Oct 14, 2019

Copy link
Copy Markdown
Member

@ambv Please, evaluate if we should merge this bugfix backport before the release or we should instead wait for 3.8.1. Check the issue discussion for more context.

@miss-islington

Copy link
Copy Markdown
Contributor Author

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

2 similar comments
@miss-islington

Copy link
Copy Markdown
Contributor Author

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

@miss-islington

Copy link
Copy Markdown
Contributor Author

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

@ambv
ambv merged commit 927f07c into python:3.8 Oct 14, 2019
@bedevere-bot

Copy link
Copy Markdown

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

@miss-islington
miss-islington deleted the backport-fd5c414-3.8 branch October 14, 2019 11:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants