Skip to content

Conversation

@ammaraskar
Copy link
Member

@ammaraskar ammaraskar commented Oct 24, 2019

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

Copy link
Contributor

@aeros aeros left a comment

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.

Copy link
Contributor

@eamanu eamanu left a comment

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

zware commented Oct 25, 2019

Thanks for the patch!

@bedevere-bot
Copy link

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