Skip to content

bpo-39868: Update Language Reference for PEP 572.#18793

Merged
gvanrossum merged 8 commits intopython:masterfrom
brandtbucher:572-docs
Mar 6, 2020
Merged

bpo-39868: Update Language Reference for PEP 572.#18793
gvanrossum merged 8 commits intopython:masterfrom
brandtbucher:572-docs

Conversation

@brandtbucher
Copy link
Copy Markdown
Member

@brandtbucher brandtbucher commented Mar 6, 2020

Leaving the prose part blank for @shankarj67 to fill in a follow-up PR.

https://bugs.python.org/issue39868

@brandtbucher brandtbucher requested a review from gvanrossum March 6, 2020 01:48
@brandtbucher
Copy link
Copy Markdown
Member Author

I think this should be good, but perhaps you can double-check my grammar here @gvanrossum? You likely have a better understanding of PEP 572's corner-cases than I do.

Copy link
Copy Markdown
Member

@gvanrossum gvanrossum left a comment

Choose a reason for hiding this comment

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

Looks good. I'll merge this now. For the TODO part we'll wait for Shankar Jha (see bpo-39868). Please give him some guidance.

: ("elif" `assignment_expression` ":" `suite`)*
: ["else" ":" `suite`]

It selects exactly one of the suites by evaluating the expressions one by one
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.

Do you think "expression" here needs updating too? It becomes a bit ugly that way, and in context it doesn't seem ambiguous.

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.

(I thought more about it and I don't think it needs updating, so I'll land now.)

@gvanrossum gvanrossum merged commit 8bae219 into python:master Mar 6, 2020
@miss-islington
Copy link
Copy Markdown
Contributor

Thanks @brandtbucher for the PR, and @gvanrossum for merging it 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖

@bedevere-bot
Copy link
Copy Markdown

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

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Mar 6, 2020
(cherry picked from commit 8bae219)

Co-authored-by: Brandt Bucher <brandtbucher@gmail.com>
@bedevere-bot
Copy link
Copy Markdown

GH-18797 is a backport of this pull request to the 3.8 branch.

miss-islington added a commit that referenced this pull request Mar 6, 2020
(cherry picked from commit 8bae219)

Co-authored-by: Brandt Bucher <brandtbucher@gmail.com>
@brandtbucher
Copy link
Copy Markdown
Member Author

Thanks, sounds good.

@brandtbucher brandtbucher deleted the 572-docs branch July 21, 2022 20:08
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants