chore: fail CodeRabbit status check when review is skipped#53
Conversation
|
Warning Ignoring CodeRabbit configuration file changes. For security, only the configuration from the base branch is applied for open source repositories. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughAdds a ChangesCodeRabbit configuration
Estimated code review effort: 1 (Trivial) | ~3 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 Checkov (3.3.8).coderabbit.yamlTraceback (most recent call last): 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 |
Summary
CodeRabbit reports a green
SUCCESScommit status even when rate limited and the PR was never actually reviewed (seen on #43: "Review rate limited"). This is misleading when used as a required status check.Adding
.coderabbit.yamlwithfail_commit_status: trueso skipped reviews fail the check instead of silently passing.Changes
.coderabbit.yaml: new file withfail_commit_status: true,request_changes_workflow: falseTest plan
failureinstead ofsuccess@coderabbitai configurationon this PR to confirm the config is picked upFixes #52