Skip to content

[workflow] fix upgrade-pylib#7211

Merged
youknowone merged 1 commit intoRustPython:mainfrom
youknowone:workflow-fix
Feb 23, 2026
Merged

[workflow] fix upgrade-pylib#7211
youknowone merged 1 commit intoRustPython:mainfrom
youknowone:workflow-fix

Conversation

@youknowone
Copy link
Member

@youknowone youknowone commented Feb 23, 2026 •
edited by coderabbitai bot
Loading

Summary by CodeRabbit

  • Chores
    • Enhanced CI/CD workflow artifact handling to support multiple patch files.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 23, 2026 •
edited
Loading

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between f478ace and e764960.

📒 Files selected for processing (1)
  • .github/workflows/upgrade-pylib.lock.yml
📝 Walkthrough

Walkthrough

The pull request modifies a GitHub Actions workflow file to expand the artifact collection pattern in the "Upload Safe Outputs" step. The patch artifact path changes from a specific filename pattern to a broader glob pattern, allowing the workflow to capture multiple matching patch files instead of a single file.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow Configuration
.github/workflows/upgrade-pylib.lock.yml		 Modified the uploaded patch artifact path from /tmp/gh-aw/aw.patch to /tmp/gh-aw/aw*.patch to use a glob pattern for capturing multiple patch files.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • Fix copilot upgrade pylib #7121: Modifies the same GitHub Actions workflow's artifact upload steps, including changes to safe-outputs and patch file handling logic.

Poem

🐰 A glob pattern hops through the night,
Where aw.patch becomes aw*.patch bright,
Catching all patches that come into sight,
Multiple files now bundled just right,
Our workflow's now flexible and light! ✨

✨ 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.

@youknowone youknowone added the skip:ci Skip running the ci label Feb 23, 2026
@youknowone youknowone marked this pull request as ready for review February 23, 2026 15:15
@youknowone youknowone merged commit 0244657 into RustPython:main Feb 23, 2026
2 checks passed
@youknowone youknowone deleted the workflow-fix branch February 23, 2026 15:15
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.

1 participant

@youknowone