◐ Shell
reader mode source ↗
Skip to content

Update test_peepholer test_compiler_codegen test_compiler_assemble from 3.14.2#6870

Merged
youknowone merged 2 commits into
RustPython:mainfrom
ShaharNaveh:update-dis-3-14-2
Jan 27, 2026
Merged

Update test_peepholer test_compiler_codegen test_compiler_assemble from 3.14.2#6870
youknowone merged 2 commits into
RustPython:mainfrom
ShaharNaveh:update-dis-3-14-2

Conversation

@ShaharNaveh

@ShaharNaveh ShaharNaveh commented Jan 25, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@coderabbitai

coderabbitai Bot commented Jan 25, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (3)
  • Lib/test/test_compiler_assemble.py is excluded by !Lib/**
  • Lib/test/test_compiler_codegen.py is excluded by !Lib/**
  • Lib/test/test_peepholer.py is excluded by !Lib/**

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

  • 🔍 Trigger a full review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@youknowone

youknowone commented Jan 25, 2026

Copy link
Copy Markdown
Member

conflict to #6869
Could you merge them into single PR? Otherwise merge one first?

@ShaharNaveh

ShaharNaveh commented Jan 25, 2026

Copy link
Copy Markdown
Contributor Author

conflict to #6869 Could you merge them into single PR? Otherwise merge one first?

Somehow I missed it 🤦
You can merge #6869 first, I'll resolve conflicts later

@github-actions

github-actions Bot commented Jan 27, 2026

Copy link
Copy Markdown
Contributor

📦 Library Dependencies

The following Lib/ modules were modified. Here are their dependencies:

[x] test: cpython/Lib/test/test_compiler_assemble.py

dependencies:

dependent tests: (no tests depend on compiler_assemble)

[x] test: cpython/Lib/test/test_compiler_codegen.py

dependencies:

dependent tests: (no tests depend on compiler_codegen)

[x] test: cpython/Lib/test/test_peepholer.py (TODO: 35)

dependencies:

dependent tests: (no tests depend on peepholer)

Legend:

  • [+] path exists in CPython
  • [x] up-to-date, [ ] outdated

@youknowone youknowone changed the title Update dis & opcode from 3.14.2 Jan 27, 2026
Hide details View details @youknowone youknowone merged commit aa99c05 into RustPython:main Jan 27, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@youknowone youknowone youknowone approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ShaharNaveh @youknowone