FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Update `test_pep646_syntax` from v3.14.2 by fanninpm · Pull Request #6928 · RustPython/RustPython · GitHub

/ RustPython Public

Update test_pep646_syntax from v3.14.2 - #6928

Merged
youknowone merged 3 commits into
RustPython:mainfrom
fanninpm:3.14-test-pep646-syntax
Feb 1, 2026
ConversationCommits (3)ChecksFiles changed
Merged

Update test_pep646_syntax from v3.14.2#6928
youknowone merged 3 commits into
RustPython:mainfrom
fanninpm:3.14-test-pep646-syntax

Conversation

fanninpm commented Feb 1, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Contributor

This test should be platform-independent, as it should only test syntax.

Summary by CodeRabbit

  • Chores
    • Expanded automated test coverage in the CI pipeline by adding PEP 646 syntax tests to the continuous integration environment test suite.

✏️ Tip: You can customize this high-level summary in your review settings.

coderabbitai Bot commented Feb 1, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

A single line is added to the CI workflow configuration to include test_pep646_syntax in the test suite, expanding the tests executed during continuous integration for PEP 646 syntax validation.

Changes

Cohort / File(s) Summary
CI Configuration
.github/workflows/ci.yaml		 Added test_pep646_syntax to test lists in both PLATFORM_INDEPENDENT_TESTS and the top-level env test group.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested reviewers

  • youknowone

Poem

A test for syntax, PEP 646's way, 🐰
Now runs through CI, come what may,
One line added, workflows gleam bright,
Our checks grow stronger with each flight! ✨

🚥 Pre-merge checks | ✅ 3 ✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title describes updating test_pep646_syntax from v3.14.2, which directly matches the main objective of adding this test to the CI environment and marking it as platform-independent.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches 🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

fanninpm mentioned this pull request Feb 1, 2026
Open

github-actions Bot commented Feb 1, 2026

Copy link
Copy Markdown
Contributor

📦 Library Dependencies

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

[ ] test: cpython/Lib/test/test_pep646_syntax.py (TODO: 12)

dependencies:

dependent tests: (no tests depend on pep646_syntax)

Legend:

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

youknowone merged commit fdf93c5 into RustPython:main Feb 1, 2026
14 checks passed
fanninpm deleted the 3.14-test-pep646-syntax branch February 1, 2026 04:22
This was referenced Feb 1, 2026
Merged
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

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

Back | FazBrowse Home | New Git URL