Skip to content

Comments

Restrict CodeRabbit trigger to devel branch only#245

Merged
tpantelis merged 1 commit intosubmariner-io:develfrom
tpantelis:cr_devel_only
Feb 13, 2026
Merged

Restrict CodeRabbit trigger to devel branch only#245
tpantelis merged 1 commit intosubmariner-io:develfrom
tpantelis:cr_devel_only

Conversation

@tpantelis
Copy link
Contributor

@tpantelis tpantelis commented Feb 13, 2026

Modified the workflow to only trigger CodeRabbit review on PRs targeting the devel branch. We don't need a CodeRabbit review on release branches since the submariner-bot is configured to auto-approve bot PRs. Doing this also reduces the chance of triggering the CodeRabbit rate limiter.

Summary by CodeRabbit

  • Chores
    • Updated code review automation workflow to selectively post reviews based on the pull request's target branch, improving development process efficiency.

Modified the workflow to only trigger CodeRabbit review on PRs
targeting the `devel` branch. We don't need a CodeRabbit review
on release branches since the `submariner-bot` is configured
to auto-approve bot PRs. Doing this also reduces the chance of
triggering the CodeRabbit rate limiter.

Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
@submariner-bot
Copy link

🤖 Created branch: z_pr245/tpantelis/cr_devel_only

@coderabbitai
Copy link

coderabbitai bot commented Feb 13, 2026

Walkthrough

A GitHub Actions workflow was modified to conditionally post CodeRabbit review comments only for pull requests targeting the devel branch, with added PR base branch verification and logging.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow
.github/workflows/coderabbit-trigger.yml
Added PR detail fetch to retrieve base branch information and conditional logic to post CodeRabbit comment only when pr.base.ref == 'devel', with logging for skipped non-devel targets.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • Oats87
  • sridhargaddam
  • skitt
🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately and concisely describes the main change: restricting the CodeRabbit trigger workflow to only run on the devel branch.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into devel
Actionable Comments Resolved ✅ Passed No unresolved actionable comments from reviewers were found in the PR context or repository, satisfying the check requirement.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


No actionable comments were generated in the recent review. 🎉


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.

❤️ Share

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

@tpantelis tpantelis enabled auto-merge (rebase) February 13, 2026 14:14
@tpantelis tpantelis merged commit d21eafc into submariner-io:devel Feb 13, 2026
8 checks passed
@submariner-bot
Copy link

🤖 Closed branches: [z_pr245/tpantelis/cr_devel_only]

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.

2 participants