Skip to content

ci: Enable merge_group trigger for merge queue support#2166

Merged
cgwalters merged 1 commit intobootc-dev:mainfrom
cgwalters:merge-queue
Apr 25, 2026
Merged

ci: Enable merge_group trigger for merge queue support#2166
cgwalters merged 1 commit intobootc-dev:mainfrom
cgwalters:merge-queue

Conversation

@cgwalters
Copy link
Copy Markdown
Collaborator

Add merge_group to the CI workflow triggers so that GitHub merge queues can be used. This is a prerequisite for enabling merge queues on the repository.

Also drop the PR number from artifact names — artifacts are already scoped to a workflow run, so the matrix parameters alone provide sufficient uniqueness.

Ref: bootc-dev/infra#143
Assisted-by: OpenCode (Claude Opus 4)

Add merge_group to the CI workflow triggers so that GitHub merge
queues can be used. This is a prerequisite for enabling merge queues
on the repository.

Also drop the PR number from artifact names — artifacts are already
scoped to a workflow run, so the matrix parameters alone provide
sufficient uniqueness.

Ref: bootc-dev/infra#143
Assisted-by: OpenCode (Claude Opus 4)
Signed-off-by: Colin Walters <walters@verbum.org>
@gemini-code-assist
Copy link
Copy Markdown
Contributor

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@cgwalters cgwalters enabled auto-merge (rebase) April 24, 2026 19:52
@bootc-bot bootc-bot Bot requested a review from ckyrouac April 24, 2026 19:52
@jmarrero
Copy link
Copy Markdown
Contributor

Maybe we should hold off for merge queue until the dust settles on the github regression that seems pretty bad. https://www.githubstatus.com/incidents/zsg1lk7w13cf

@cgwalters cgwalters disabled auto-merge April 25, 2026 17:14
@cgwalters cgwalters merged commit 2829403 into bootc-dev:main Apr 25, 2026
60 of 66 checks passed
@cgwalters
Copy link
Copy Markdown
Collaborator Author

Maybe we should hold off for merge queue until the dust settles on the github regression that seems pretty bad.

Yeah that came up in bootc-dev/infra#143 (comment)

But...they say they fixed it too. I dunno. I'd still lean towards trying it.

Among other things, in the situation we're in now I think it means we don't need to ensure each PR gets rebased on main to pick up the test-coreos job change, which alone is quite nice.

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.

3 participants