NO-ISSUE: Require osac pre-commit, unit, and integration checks - #191
Conversation
Unit, integration, and pre-commit jobs already run on PRs but only e2e gates and check-labels block merge.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository: osac-project/coderabbit/.coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan includes up to 12 reviews per rolling hour; 11 remain after this review. WalkthroughThe ChangesOSAC repository protection
Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: 🔵 Low · up to This PR adds existing osac CI checks as merge requirements. It is mergeable with owner awareness, but the exact check names and skipped-job behavior should be confirmed before applying the rule to avoid unexpectedly blocking valid merges or weakening enforcement. Suggested reviewers: 🚥 Pre-merge checks | ✅ 11✅ Passed checks (11 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Summary
pre-commit, unit tests, integration tests) torepo_osacrequired_status_checksso failures block merge.if:skips already report success, so docs-only PRs stay mergeable.Test plan
tofu planshows onlyrepo_osacrequired-check additionsSummary by CodeRabbit