Skip to content

bpo-46725: Document starred expressions in for statements - #31481

Merged
miss-islington merged 4 commits into
python:mainfrom
pablogsal:bpo-46725
Feb 22, 2022
Merged

bpo-46725: Document starred expressions in for statements#31481
miss-islington merged 4 commits into
python:mainfrom
pablogsal:bpo-46725

Conversation

@pablogsal

@pablogsalpablogsal commented Feb 21, 2022•
edited by miss-islington
Loading

Copy link
Copy Markdown
Member

https://bugs.python.org/issue46725

Automerge-Triggered-By: GH:pablogsal

@bedevere-botbedevere-bot added the docs Documentation in the Doc dir label Feb 21, 2022
JelleZijlstra
JelleZijlstra reviewed Feb 21, 2022
View reviewed changes

@JelleZijlstraJelleZijlstra left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we also add unit tests that test this behavior?

Comment threadDoc/reference/compound_stmts.rst
gvanrossum
gvanrossum reviewed Feb 22, 2022
View reviewed changes
Comment threadLib/test/test_grammar.py Outdated
@pablogsal@gvanrossum
Update Lib/test/test_grammar.py
b052610
Co-authored-by: Guido van Rossum <gvanrossum@gmail.com>
gvanrossum
gvanrossum reviewed Feb 22, 2022
View reviewed changes
Comment threadDoc/whatsnew/3.11.rst Outdated
@pablogsal@gvanrossum
Update Doc/whatsnew/3.11.rst
8efad04
Co-authored-by: Guido van Rossum <gvanrossum@gmail.com>
@miss-islington

miss-islington commented Feb 22, 2022

Copy link
Copy Markdown
Contributor

Sorry, I can't merge this PR. Reason: 4 of 6 required status checks are expected..

@miss-islington

miss-islington commented Feb 22, 2022

Copy link
Copy Markdown
Contributor

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

@miss-islington
miss-islington merged commit 7fb94fd into python:mainFeb 22, 2022
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@JelleZijlstraJelleZijlstraJelleZijlstra left review comments
@gvanrossumgvanrossumgvanrossum approved these changes

Assignees

No one assigned

Labels

docsDocumentation in the Doc dirskip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@pablogsal@miss-islington@JelleZijlstra@gvanrossum@the-knights-who-say-ni@bedevere-bot