Skip to content

ci: trigger CI on merge_group events#204

Open
pgherveou wants to merge 1 commit into
mainfrom
ci/merge-group-trigger
Open

ci: trigger CI on merge_group events#204
pgherveou wants to merge 1 commit into
mainfrom
ci/merge-group-trigger

Conversation

@pgherveou
Copy link
Copy Markdown
Collaborator

Problem

The branch ruleset on main requires the "CI Status" check and enables a merge queue. When a PR is enqueued, GitHub builds a temporary merge commit on a gh-readonly-queue/main/... branch and fires a merge_group event, expecting CI to run against it and report "CI Status".

ci.yml only triggered on pull_request, push to main, and workflow_dispatch — never merge_group. So nothing ran against the queue's temp commit, its combined status stayed pending, and any enqueued PR sat in AWAITING_CHECKS until the queue's 60-minute timeout, then dequeued. The merge queue could never merge anything.

Fix

Add the merge_group event to ci.yml's triggers so the full CI suite (and therefore the "CI Status" gate) runs against the queue's merge commit.

The concurrency group keys on github.ref, which is distinct per merge-group branch, so queue runs don't cancel PR runs. The disabled e2e job reports skipped, which the ci-status gate already accepts.

Note

This PR itself must reach main before the queue works. Since the queue is currently non-functional, a maintainer will need to merge it directly (bypass) or temporarily disable the merge-queue rule.

The branch ruleset requires the "CI Status" check and enables a merge
queue. The queue builds a temporary merge commit and fires a merge_group
event, but ci.yml did not listen for it, so CI never ran against the
queue's commit and the required check stayed pending. Add the merge_group
trigger so the queue can resolve its required check.
@pgherveou pgherveou requested a review from a team June 5, 2026 20:30
@filvecchiato filvecchiato enabled auto-merge June 5, 2026 20:50
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