Skip to content

ci: run the checks for merge queue groups - #176

Merged
kvlonge merged 1 commit into
mainfrom
ci/merge-queue
Nov 11, 2026
Merged

kvlonge merged 1 commit into
mainfrom
ci/merge-queue

Conversation

@kvlonge

@kvlonge kvlonge commented Nov 11, 2026

Copy link
Copy Markdown
Contributor

Why

SQLQuack's main requires branches to be up to date before merging. Each merge forces every other open pull request to update and rerun its ~30-minute CI. A merge queue tests each pull request on top of the latest main and merges it, with no manual updates. The queue needs the required Verify check to run on merge_group events; today CI runs only on pull_request, schedule and dispatch, so a queued pull request would wait forever. Pull Request Metadata and the version guard already handle merge_group.

Changes

  • ci.yml also runs on merge_group.
  • A merge group is scoped like a pull request: select_replay scope and EVIDENCE_BASE use the group's base_sha.
  • Like a pull request, a merge group only reads the Rust and replay caches and skips the nightly-only DuckShop and fuzz jobs.

The ruleset switch from "require up to date" to the merge queue is a separate repository setting, applied after this merges.

Verification

  • actionlint 1.7.7 passes on the changed workflow (and on main's).
  • ci.yml is a full-replay path, so this pull request's own run replays every case.

The merge queue tests each pull request on top of the latest main before it
lands. Verify is the required check, so CI must also run on merge_group
events, or a queued pull request would wait forever.

A merge group is scoped like a pull request, against the group's base
commit, and like a pull request it only reads the caches and skips the
nightly-only DuckShop and fuzz jobs.
@kvlonge
kvlonge enabled auto-merge (squash) November 11, 2026 09:58
@kvlonge
kvlonge merged commit 11de3d7 into main Nov 11, 2026
8 checks passed
@kvlonge
kvlonge deleted the ci/merge-queue branch November 11, 2026 10:39
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