Skip to content

ci: require CodeRabbit review check for enhancement proposals - #235

Closed
minmzzhang wants to merge 1 commit into
osac-project:mainfrom
minmzzhang:require-coderabbit-review-check
Closed

minmzzhang wants to merge 1 commit into
osac-project:mainfrom
minmzzhang:require-coderabbit-review-check

Conversation

@minmzzhang

Copy link
Copy Markdown
Contributor

Summary

Require the existing CodeRabbit status check for enhancement-proposals before merging.

The repository continues to require two human approvals. This adds automated review completion as a separate required status check; it does not count CodeRabbit as a human approval.

Test plan

  • git diff --check
  • Run tofu fmt -check
  • Run tofu plan
  • Confirm the applied ruleset requires CodeRabbit
  • Confirm a PR with a failing or missing CodeRabbit check cannot merge

Assisted-by: Codex noreply@openai.com

Require the existing CodeRabbit status check for enhancement-proposals so automated review completes before merge. This supplements the two human approvals and human-review gate.

Assisted-by: Codex <noreply@openai.com>
@coderabbitai

coderabbitai Bot commented Sep 24, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

Comment @coderabbitai help to get the list of available commands.

@minmzzhang

Copy link
Copy Markdown
Contributor Author

Closing this draft because CodeRabbit should not be a required merge check.

@minmzzhang minmzzhang closed this Sep 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant