Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

gh-136438: Make suretest_ast pass with all optimization levels#136596

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
Eclips4 merged 3 commits intopython:mainfromefimov-mikhail:issue-136438-test_ast
Jul 18, 2025

Conversation

efimov-mikhail
Copy link
Contributor

@efimov-mikhailefimov-mikhail commentedJul 12, 2025
edited by bedevere-appbot
Loading

Now tests pass with all combinations of -OO and --without-doc-strings.
It was long lists of errors before, but all proposed fixes are just to add some neededoptimize=False options.

@bedevere-appbedevere-appbot mentioned this pull requestJul 12, 2025
8 tasks
@bedevere-appbedevere-appbot added the testsTests in the Lib/test dir labelJul 12, 2025
@Eclips4
Copy link
Member

I believe the explanation is as follows:
If theoptimize parameter isn’t passed explicitly, the interpreter uses its internal setting (which can be modified using the -O or-OO flags).
Am I right,@efimov-mikhail?

Co-authored-by: Kirill Podoprigora <kirill.bast9@mail.ru>
@efimov-mikhail
Copy link
ContributorAuthor

efimov-mikhail commentedJul 17, 2025
edited
Loading

If theoptimize parameter isn’t passed explicitly, the interpreter uses its internal setting (which can be modified using the -O or-OO flags).

Exactly. I've added checks only in those tests that fail with-OO. We now force to runast.parse andcompile without optimizations to be sure that their results don't depend on runtime flags.

Eclips4 reacted with thumbs up emoji

@Eclips4Eclips4 merged commit03017a8 intopython:mainJul 18, 2025
40 checks passed
@Eclips4
Copy link
Member

Thank you@efimov-mikhail!

efimov-mikhail reacted with heart emoji

@efimov-mikhail
Copy link
ContributorAuthor

Thx for merge,@Eclips4 !

Eclips4 reacted with rocket emoji

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

@Eclips4Eclips4Eclips4 left review comments

@tomasr8tomasr8Awaiting requested review from tomasr8tomasr8 is a code owner

@sobolevnsobolevnAwaiting requested review from sobolevn

Assignees
No one assigned
Labels
skip newstestsTests in the Lib/test dir
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

3 participants
@efimov-mikhail@Eclips4@brianschubert

[8]ページ先頭

©2009-2025 Movatter.jp