8000 gh-127975: Avoid reusing quote types in ast.unparse if not needed by hauntsaninja · Pull Request #127980 · python/cpython · GitHub
[go: up one dir, main page]
Skip to content

gh-127975: Avoid reusing quote types in ast.unparse if not needed #127980

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
merged 4 commits into from
Jan 31, 2025
Merged

gh-127975: Avoid reusing quote types in ast.unparse if not needed #127980

merged 4 commits into from
Jan 31, 2025

Conversation

hauntsaninja
Copy link
Contributor
@hauntsaninja hauntsaninja commented Dec 16, 2024 •
edited by bedevere-app bot
Loading

Pablo was curious about a PR, so opening this as a draft

@bedevere-app bedevere-app bot mentioned this pull request Dec 16, 2024
Closed
@hauntsaninja hauntsaninja marked this pull request as ready for review December 20, 2024 08:43
Eclips4
Eclips4 approved these changes Dec 20, 2024
View reviewed changes
@hauntsaninja hauntsaninja enabled auto-merge (squash) January 31, 2025 06:52
@hauntsaninja hauntsaninja merged commit 8df5193 into python:main Jan 31, 2025
39 checks passed
@hauntsaninja hauntsaninja deleted the gh-127975 branch February 2, 2025 22:41
@hauntsaninja hauntsaninja added needs backport to 3.12 only security fixes needs backport to 3.13 bugs and security fixes labels Feb 2, 2025
@miss-islington-app
Copy link

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

@miss-islington-app
Copy link

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

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Feb 2, 2025
@hauntsaninja @miss-islington
pythongh-127975: Avoid reusing quote types in ast.unparse if not need…
0d5ae2d 
…ed (pythonGH-127980)

(cherry picked from commit 8df5193)

Co-authored-by: Shantanu <12621235+hauntsaninja@users.noreply.github.com>
@bedevere-app
Copy link
bedevere-app bot commented Feb 2, 2025

GH-129600 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 2, 2025
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Feb 2, 2025
@hauntsaninja @miss-islington
pythongh-127975: Avoid reusing quote types in ast.unparse if not need…
9f97d59 
…ed (pythonGH-127980)

(cherry picked from commit 8df5193)

Co-authored-by: Shantanu <12621235+hauntsaninja@users.noreply.github.com>
@bedevere-app
Copy link
bedevere-app bot commented Feb 2, 2025

GH-129601 is a backport of this pull request to the 3.12 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.12 only security fixes label Feb 2, 2025
hauntsaninja added a commit that referenced this pull request Feb 2, 2025
@miss-islington @hauntsaninja
[3.12] gh-127975: Avoid reusing quote types in ast.unparse if not nee…
9709746 
…ded (GH-127980) (#129601)

gh-127975: Avoid reusing quote types in ast.unparse if not needed (GH-127980)
(cherry picked from commit 8df5193)

Co-authored-by: Shantanu <12621235+hauntsaninja@users.noreply.github.com>
hauntsaninja added a commit that referenced this pull request Feb 3, 2025
@miss-islington @hauntsaninja
[3.13] gh-127975: Avoid reusing quote types in ast.unparse if not nee…
b13b76f 
…ded (GH-127980) (#129600)

gh-127975: Avoid reusing quote types in ast.unparse if not needed (GH-127980)
(cherry picked from commit 8df5193)

Co-authored-by: Shantanu <12621235+hauntsaninja@users.noreply.github.com>
srinivasreddy pushed a commit to srinivasreddy/cpython that referenced this pull request Feb 7, 2025
@hauntsaninja @srinivasreddy
pythongh-127975: Avoid reusing quote types in ast.unparse if not need…
bd7cdd3 
…ed (python#127980)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JelleZijlstra JelleZijlstra JelleZijlstra approved these changes

@Eclips4 Eclips4 Eclips4 approved these changes

@isidentical isidentical Awaiting requested review from isidentical isidentical is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@hauntsaninja @JelleZijlstra @Eclips4
0