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

auto mark parent tests by youknowone · Pull Request #6778 · RustPython/RustPython · GitHub

/ RustPython Public

auto mark parent tests - #6778

Merged
youknowone merged 2 commits into
RustPython:mainfrom
youknowone:auto-mark-parent
Jan 18, 2026
ConversationCommits (2)ChecksFiles changed
Merged

auto mark parent tests#6778
youknowone merged 2 commits into
RustPython:mainfrom
youknowone:auto-mark-parent

Conversation

youknowone commented Jan 18, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Member

also fix #6721

Summary by CodeRabbit

  • Chores
    • Auto-format workflow now runs unconditionally on all PRs, regardless of label configuration
    • Enhanced test marking script with inheritance-aware method detection for improved accuracy in identifying test methods
    • Optimized library updater performance through pre-populated caching strategy

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

coderabbitai Bot commented Jan 18, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

This pull request modifies CI/CD automation and test infrastructure across three files: removing a label-based conditional guard from the auto-format workflow, enhancing test failure detection with AST-based inheritance analysis, and optimizing cache initialization in patch line processing.

Changes

Cohort / File(s) Summary
CI/CD Workflow
.github/workflows/pr-auto-commit.yaml		 Removed conditional guard preventing auto_format job execution when PR has skip:ci label; job now runs unconditionally
Test Automation
scripts/auto_mark_test.py		 Enhanced test pattern detection from "Run tests? sequentially" to "Run <n> tests? sequentially"; added build_inheritance_info() and find_method_definition() helpers for AST-based inheritance analysis; modified remove_expected_failures() to resolve method definitions across class inheritance hierarchies
Patch Processing
scripts/lib_updater.py		 Refactored cache initialization in iter_patch_lines() to pre-populate class end lines before Phase 1 processing, removing incremental cache updates during iteration

Possibly related PRs

  • RustPython/RustPython#6706: Modifies the same .github/workflows/pr-auto-commit.yaml workflow for formatting/commit step execution control

Poem

🐰 The workflows now run without delay,
While tests trace inheritance their way,
With caches built before the race,
Each change finds its rightful place! ✨

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

✨ Finishing touches
  • 📝 Generate docstrings

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 added the skip:ci Skip running the ci label Jan 18, 2026

github-actions Bot commented Jan 18, 2026

Copy link
Copy Markdown
Contributor

Code has been automatically formatted

The code in this PR has been formatted using:

  • ruff format
    Please pull the latest changes before pushing again:
git pull origin auto-mark-parent

youknowone marked this pull request as ready for review January 18, 2026 13:06
youknowone merged commit 2b8fac3 into RustPython:main Jan 18, 2026
8 of 9 checks passed
youknowone deleted the auto-mark-parent branch January 18, 2026 13:06
coderabbitai Bot mentioned this pull request Feb 1, 2026
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

No reviews

Assignees

No one assigned

Labels

skip:ci Skip running the ci

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

auto-format must not be blocked by skip:ci label

1 participant

Back | FazBrowse Home | New Git URL