Skip to content

bpo-38558: Mention := in conditions tutorial.#16919

Merged
zware merged 2 commits into
python:masterfrom
ammaraskar:bpo-38558
Oct 25, 2019
Merged

bpo-38558: Mention := in conditions tutorial.#16919
zware merged 2 commits into
python:masterfrom
ammaraskar:bpo-38558

Conversation

@ammaraskar

@ammaraskar ammaraskar commented Oct 24, 2019

Copy link
Copy Markdown
Member

This mention about assignments is now stale with the introduction of the assignment expression operator. Adjust the wording to mention that := must be used explicitly, the rest of the paragraph explaining motivation still applies.

https://bugs.python.org/issue38558

@aeros aeros left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Good catch @ammaraskar, this section of the tutorial should definitely be updated to mention the walrus operator for 3.8 and 3.9.

Comment thread Doc/tutorial/datastructures.rst Outdated

@eamanu eamanu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM after apply @aeros suggestion

@zware
zware merged commit cb2cf06 into python:master Oct 25, 2019
@miss-islington

Copy link
Copy Markdown
Contributor

Thanks @ammaraskar for the PR, and @zware for merging it 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖 I'm not a witch! I'm not a witch!

@zware

zware commented Oct 25, 2019

Copy link
Copy Markdown
Member

Thanks for the patch!

@bedevere-bot

Copy link
Copy Markdown

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

miss-islington added a commit that referenced this pull request Oct 25, 2019
(cherry picked from commit cb2cf06)

Co-authored-by: Ammar Askar <ammar@ammaraskar.com>
jacobneiltaylor pushed a commit to jacobneiltaylor/cpython that referenced this pull request Dec 5, 2019
shihai1991 pushed a commit to shihai1991/cpython that referenced this pull request Jan 31, 2020
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.

8 participants