Skip to content

bpo-34013: Don't consider a grouped expression when reporting legacy print syntax errors - #27521

Merged
pablogsal merged 1 commit into
python:mainfrom
pablogsal:bpo-34013
Aug 1, 2021
Merged

pablogsal merged 1 commit into
python:mainfrom
pablogsal:bpo-34013

Conversation

@pablogsal

@pablogsal pablogsal commented Jul 31, 2021

Copy link
Copy Markdown
Member

@pablogsal

Copy link
Copy Markdown
Member Author

@brandtbucher Could you review this at the earliest convenience? I would like to land this before the end of tomorrow so I can release the rc on schedule.

@brandtbucher brandtbucher 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.

Thanks for the quick fix!

@pablogsal
pablogsal merged commit 208a7e9 into python:main Aug 1, 2021
@pablogsal
pablogsal deleted the bpo-34013 branch August 1, 2021 01:10
@miss-islington

Copy link
Copy Markdown
Contributor

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

@bedevere-bot

Copy link
Copy Markdown

GH-27522 is a backport of this pull request to the 3.10 branch.

@bedevere-bot bedevere-bot removed the needs backport to 3.10 only security fixes label Aug 1, 2021
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Aug 1, 2021
…print syntax errors (pythonGH-27521)

(cherry picked from commit 208a7e9)

Co-authored-by: Pablo Galindo Salgado <Pablogsal@gmail.com>
miss-islington added a commit that referenced this pull request Aug 1, 2021
…print syntax errors (GH-27521)

(cherry picked from commit 208a7e9)

Co-authored-by: Pablo Galindo Salgado <Pablogsal@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants