Skip to content

gh-144140: Optimize len for string constants in optimizer #144142

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
Open
y1yang0 wants to merge 2 commits into
base: main
Choose a base branch
from
Open

gh-144140: Optimize len for string constants in optimizer #144142

y1yang0 wants to merge 2 commits into from
+45 −6

Conversation

@y1yang0
Copy link

@y1yang0 y1yang0 commented Jan 22, 2026 •
edited by bedevere-app bot
Loading

@python-cla-bot
Copy link

python-cla-bot bot commented Jan 22, 2026 •
edited
Loading

All commit authors signed the Contributor License Agreement.

CLA signed

@bedevere-app
Copy link

bedevere-app bot commented Jan 22, 2026

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@bedevere-app bedevere-app bot mentioned this pull request Jan 22, 2026
Open
@bedevere-app
Copy link

bedevere-app bot commented Jan 22, 2026

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@y1yang0
Copy link
Author

y1yang0 commented Jan 22, 2026

I'm not sure if this patch requires a new news item. I'll let the reviewer decide.

@aisk aisk added the skip news label Jan 22, 2026
@aisk
Copy link
Contributor

aisk commented Jan 22, 2026

Hi, I think there have been some CI changes in the main branch, and this branch is not up to date, which is causing the CI to fail. Could you please sync this branch with main (you can simply click the “Update branch” button above, and please avoid rebasing and force push in the CPython repo) to make the CI pass?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

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

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

@Fidget-Spinner Fidget-Spinner Awaiting requested review from Fidget-Spinner Fidget-Spinner is a code owner

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

Assignees

No one assigned

Labels

awaiting review skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@y1yang0 @aisk