Skip to content

Conversation

@pablogsal
Copy link
Member

@pablogsal pablogsal commented Jun 23, 2019

@pablogsal pablogsal merged commit b51b713 into python:master Jun 25, 2019
@miss-islington
Copy link
Contributor

Thanks @pablogsal for the PR 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖

@pablogsal pablogsal deleted the bpo35224-2 branch June 25, 2019 01:42
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jun 25, 2019
…nsions (pythonGH-14319)

(cherry picked from commit b51b713)

Co-authored-by: Pablo Galindo <Pablogsal@gmail.com>
@bedevere-bot
Copy link

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

miss-islington added a commit that referenced this pull request Jun 25, 2019
…nsions (GH-14319)

(cherry picked from commit b51b713)

Co-authored-by: Pablo Galindo <Pablogsal@gmail.com>
* The :opcode:`MAP_ADD` now expects the value as the first element in the
stack and the key as the second element. This change was made so the key
is always evaluated before the value in dictionary comprehensions, as
porposed by :pep:`572`. (Contributed by Jörn Heissler in :issue:`35224`.)
Copy link
Member

Choose a reason for hiding this comment

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

porposed -> proposed

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed in #14371

lisroach pushed a commit to lisroach/cpython that referenced this pull request Sep 10, 2019
DinoV pushed a commit to DinoV/cpython that referenced this pull request Jan 14, 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.

6 participants