ci: add merge_group trigger to required-check workflows#40
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (5)
WalkthroughFive GitHub Actions workflows ( Changesmerge_group Trigger Addition
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Suggested labels
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Scanned FilesNone |
Summary
merge_group:to workflows emitting required status checks so they fire in the merge queueGitHub merge queues fire a
merge_groupevent, notpull_request. Without the trigger,required checks never report and the queue stalls indefinitely waiting for them.
Workflows fixed:
security-analysis.yml(emits: Security Gate Validation)pr-validation.yml(emits: Dependency & Standards Validation)reuse.yml(emits: Check REUSE Compliance)ci.yml(emits: CI Gate)sonarcloud.yml(emits: SonarCloud Analysis)Test plan
Generated with Claude Code
Summary by CodeRabbit