Skip to content

gh-133194: Fix regression with PEP 758 parsing on older feature_version#133289

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.

Sign up for GitHub

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

Jump to bottom
Merged
sobolevn merged 1 commit into from
May 3, 2025
Merged

gh-133194: Fix regression with PEP 758 parsing on older feature_version#133289

sobolevn merged 1 commit into from
May 3, 2025

Conversation

@sobolevn
Copy link
Member

@sobolevnsobolevn commented May 2, 2025•
edited
Loading

My previous change was not quite correct in two ways:

  • minor: missing "are" in the exception message
  • major: it stopped allowing parsing valid code like except TypeError: with older feature_versions :(

I added a lot more tests and fixed the problem (hopefully)! Reviews are highly appreciated 🙏

Fixes#133195
Fixes#133194
cc @cdce8p

@sobolevnsobolevn changed the title gh-133192: Fix regression with PEP 758 parsing on older feature_versiongh-133194: Fix regression with PEP 758 parsing on older feature_versionMay 2, 2025
@bedevere-appbedevere-appbot mentioned this pull request May 2, 2025
Closed
cdce8p
cdce8p approved these changes May 2, 2025
View reviewed changes
Copy link
Contributor

@cdce8pcdce8p left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks @sobolevn for picking this up so quickly 🙏🏻

lysnikolaou
lysnikolaou approved these changes May 2, 2025
View reviewed changes
Copy link
Member

@lysnikolaoulysnikolaou left a comment

Choose a reason for hiding this comment

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

Yeah, this looks good to me! Missed it in the earlier PR. Surprised that the tests didn't catch it.

@sobolevn
Copy link
MemberAuthor

sobolevn commented May 2, 2025

I will keep this open for a day, so maybe @pablogsal can also comment.

Thanks everyone 👍

@sobolevnsobolevn merged commit ca0a96d into python:mainMay 3, 2025
49 checks passed
Pranjal095 pushed a commit to Pranjal095/cpython that referenced this pull request Jul 12, 2025
@sobolevn@Pranjal095
pythongh-133194: Fix regression with PEP 758 parsing on older `featur…
75e6788
…e_version` (python#133289) pythongh-133192: Fix regression with PEP 758 parsing on older `feature_version`
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lysnikolaoulysnikolaoulysnikolaou approved these changes

@Eclips4Eclips4Eclips4 approved these changes

@pablogsalpablogsalAwaiting requested review from pablogsalpablogsal is a code owner

+1 more reviewer

@cdce8pcdce8pcdce8p approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

PEP 758 changes do not have CHECK_VERSION guard for usage with ast.parse(..., feature_version=(3, 13))

4 participants

@sobolevn@lysnikolaou@cdce8p@Eclips4