X Tutup
Skip to content

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

Merged
ambv merged 1 commit intopython: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 intopython:3.8from
miss-islington:backport-fd5c414-3.8

Conversation

@miss-islington
Copy link
Copy Markdown
Contributor

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

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

pablogsal commented Oct 14, 2019

@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

X Tutup