Native triggers for workflow dependency#6253
Native triggers for workflow dependency#6253ShaharNaveh wants to merge 1 commit intoRustPython:mainfrom
Conversation
|
Caution Review failedThe pull request is closed. WalkthroughThe CI workflow now includes explicit push (main, release branches) and pull_request event triggers alongside existing merge_group and workflow_dispatch triggers. The PR auto-commit workflow transitions from direct pull_request_target triggers to a workflow_run trigger that activates after CI completion, with a conditional guard ensuring execution only for pull requests with successful CI runs. Changes
Sequence DiagramsequenceDiagram
participant Developer
participant PR as Pull Request
participant CI as CI Workflow
participant AutoCommit as PR Auto-Commit<br/>Workflow
participant Repo as Repository
Developer->>PR: Push commit or open PR
PR->>CI: Trigger on push/pull_request events
activate CI
CI->>CI: Run tests, checks
CI-->>Repo: Report completion status
deactivate CI
Note over AutoCommit: Listens for CI<br/>workflow_run completion
CI-->>AutoCommit: Trigger workflow_run (on success)
activate AutoCommit
AutoCommit->>AutoCommit: Guard: Check if event<br/>is pull_request<br/>and CI succeeded
AutoCommit->>AutoCommit: Run cargo fmt
AutoCommit->>Repo: Commit & push formatted changes
deactivate AutoCommit
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes
Possibly related PRs
Suggested reviewers
Poem
✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: Path: .coderabbit.yml Review profile: CHILL Plan: Pro 📒 Files selected for processing (2)
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. Comment |
|
Oh, I need to try this on my main branch first. closing for now |
No idea if it works, let's see
Summary by CodeRabbit