Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

gh-94947: Disallow parsing walrus with feature_version < (3, 8) #94948

Merged
merged 3 commits into from
Jul 18, 2022

Conversation

hauntsaninja
Copy link
Contributor

@hauntsaninja hauntsaninja commented Jul 18, 2022

@cpython-cla-bot
Copy link

cpython-cla-bot bot commented Jul 18, 2022

All commit authors signed the Contributor License Agreement.
CLA signed

@pablogsal pablogsal merged commit ae0be5a into python:main Jul 18, 2022
@miss-islington
Copy link
Contributor

Thanks @hauntsaninja for the PR, and @pablogsal for merging it 🌮🎉.. I'm working now to backport this PR to: 3.10, 3.11.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jul 18, 2022
…pythonGH-94948)

* pythongh-94947: Disallow parsing walrus with feature_version < (3, 8)

* oops, commit the parser

* 📜🤖 Added by blurb_it.

Co-authored-by: blurb-it[bot] <43283697+blurb-it[bot]@users.noreply.github.com>
(cherry picked from commit ae0be5a)

Co-authored-by: Shantanu <[email protected]>
@miss-islington
Copy link
Contributor

Sorry, @hauntsaninja and @pablogsal, I could not cleanly backport this to 3.10 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker ae0be5a53bb4caee3de4888341addd9c94133f2d 3.10

@bedevere-bot
Copy link

GH-94953 is a backport of this pull request to the 3.11 branch.

@pablogsal
Copy link
Member

@hauntsaninja Could you do the 3.10 manual backport?

miss-islington added a commit that referenced this pull request Jul 18, 2022
…4948)

* gh-94947: Disallow parsing walrus with feature_version < (3, 8)

* oops, commit the parser

* 📜🤖 Added by blurb_it.

Co-authored-by: blurb-it[bot] <43283697+blurb-it[bot]@users.noreply.github.com>
(cherry picked from commit ae0be5a)

Co-authored-by: Shantanu <[email protected]>
@hauntsaninja hauntsaninja deleted the gh-94947 branch July 18, 2022 16:32
hauntsaninja added a commit to hauntsaninja/cpython that referenced this pull request Jul 18, 2022
… (3, 8) (pythonGH-94948)

* pythongh-94947: Disallow parsing walrus with feature_version < (3, 8)

* oops, commit the parser

* 📜🤖 Added by blurb_it.

Co-authored-by: blurb-it[bot] <43283697+blurb-it[bot]@users.noreply.github.com>.
(cherry picked from commit ae0be5a)

Co-authored-by: Shantanu <[email protected]>
@bedevere-bot
Copy link

GH-94969 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 Jul 18, 2022
@hauntsaninja
Copy link
Contributor Author

@pablogsal Done!

pablogsal pushed a commit that referenced this pull request Jul 18, 2022
GH-94948) (#94969)

* gh-94947: Disallow parsing walrus with feature_version < (3, 8)

* oops, commit the parser

* 📜🤖 Added by blurb_it.

Co-authored-by: blurb-it[bot] <43283697+blurb-it[bot]@users.noreply.github.com>.
(cherry picked from commit ae0be5a)

Co-authored-by: Shantanu <[email protected]>
hauntsaninja added a commit to hauntsaninja/mypy that referenced this pull request Jul 28, 2022
As a result of python/cpython#94948
we started to fail these tests on Python 3.11b5
JelleZijlstra pushed a commit to python/mypy that referenced this pull request Jul 28, 2022
As a result of python/cpython#94948
we started to fail these tests on Python 3.11b5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants