Skip to content

Conversation

@tomasr8
Copy link
Member

@tomasr8 tomasr8 commented Feb 7, 2025

This suppresses a test case which emits SyntaxWarning

Copy link

@auvipy auvipy left a comment

Choose a reason for hiding this comment

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

If the suppression is intended, then the PR seems good to me

@pablogsal pablogsal enabled auto-merge (squash) February 13, 2025 00:15
@pablogsal pablogsal merged commit 2dd0188 into python:main Feb 13, 2025
38 checks passed
@pablogsal pablogsal added the needs backport to 3.13 bugs and security fixes label Feb 13, 2025
@miss-islington-app
Copy link

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

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Feb 13, 2025
…29830)

Suppress SyntaxWarning in test_fstring
(cherry picked from commit 2dd0188)

Co-authored-by: Tomas R. <[email protected]>
@bedevere-app
Copy link

bedevere-app bot commented Feb 13, 2025

GH-130068 is a backport of this pull request to the 3.13 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.13 bugs and security fixes label Feb 13, 2025
pablogsal pushed a commit that referenced this pull request Feb 13, 2025
#130068)

gh-129693: Suppress `SyntaxWarning` in test_fstring (GH-129830)

Suppress SyntaxWarning in test_fstring
(cherry picked from commit 2dd0188)

Co-authored-by: Tomas R <[email protected]>
@tomasr8 tomasr8 deleted the fstring-warning branch February 13, 2025 07:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip news tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants