Skip to content

ci: run CI on merge_group events - #15

Merged
jamesarich merged 1 commit into
mainfrom
ci/merge-queue
Oct 3, 2026
Merged

jamesarich merged 1 commit into
mainfrom
ci/merge-queue

Conversation

@jamesarich

@jamesarich jamesarich commented Oct 3, 2026 •

Copy link
Copy Markdown
Collaborator

Lets a merge queue run CI on the group it is about to merge, so the queue can be turned on for main.

Summary by CodeRabbit

  • Chores
    • Automated checks now also run when changes are processed through GitHub’s merge queue. This gives queued changes the same validation as other updates before they are merged, helping maintain consistent quality across the project. No changes to the app’s features or behavior are included.

Signed-off-by: James Rich <2199651+jamesarich@users.noreply.github.com>
@coderabbitai

coderabbitai Bot commented Oct 3, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 5f33f4b6-4469-4456-ba10-bd8ba3e64196
📥 Commits

Reviewing files that changed from the base of the PR and between e607914 and e5fe8a7.

📒 Files selected for processing (1)
  • .github/workflows/ci.yml

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The CI workflow now runs for GitHub merge-group events.

Changes

CI workflow

Layer / File(s) Summary
Add merge-group trigger
.github/workflows/ci.yml
The workflow adds the merge_group event as a trigger.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Feature

Merge Risk: ⚪ Minimal · up to e5fe8

The CI workflow is configured to run on merge-group events, and no concrete merge-readiness risk is evident.

Architecture Summary

Architecture risk: 🔵 Low · up to e5fe8

The changed surface does not map to a changed system, dependency edge, entrypoint, or external dependency.

Changed systems: None identified.

Architecture concerns
No architecture-level concerns identified.

Review details

Before / after behavior

  • observed — Modified behavior in .github/workflows/ci.yml: The workflow adds the merge_group event as a trigger.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the change: running CI on GitHub merge_group events.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

A rabbit checks the CI gate,
A merge-group joins the queue.
The workflow wakes to greet it,
Then runs its tasks anew.
I nibble greens and hop along,
While checks complete on cue.

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

@jamesarich
jamesarich merged commit 68cbfef into main Oct 3, 2026
9 checks passed
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