Skip to content

ci(mergify): extend the org merge-queue policy and name this repo's checks - #12

Open
ryanleecode wants to merge 4 commits into
mainfrom
ci/mergify-config
Open

ryanleecode wants to merge 4 commits into
mainfrom
ci/mergify-config

Conversation

@ryanleecode

Copy link
Copy Markdown
Contributor

Adds .mergify.yml: extends: .github (systemfsoftware/.github holds the org policy) plus one default queue rule naming this repo's checks.

  • queue_conditions (on the PR): checks / the gate; the changeset intent check (success or skipped); Same bytes on every host.
  • merge_conditions (on Mergify's queue draft PR): checks / the gate; Same bytes on every host.
  • Validated against Mergify's published JSON schema. The config does nothing until the Mergify App has this repo and .github has the shared config.
  • CI runs up to 40 min; a queue-only heavy lane is a later option.

🤖 Generated with Claude Code

ryanleecode and others added 2 commits October 18, 2026 02:14
…hecks

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

@kiro-systemf kiro-systemf Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The or: check-success = review-gate condition names a check this repository does not produce. The checks on this Very PR and on #11 are: Output bytes on arm64, Output bytes on x86_64, Same bytes on every host, check / a publishable-package change needs an intent, checks / build · attw, checks / lint, checks / plan · test jobs from recorded timings, checks / record test timings, checks / test · api-extractor, api-extractor-e2e, checks / the gate, checks / typecheck, systemfsoftware tarballs from the flake — no review-gate. As written the check can never succeed, and the queue condition would rest on the approved-reviews-by branch alone. Either drop the branch or name the check that actually carries the review gate here.

…view gate

Dependabot patch and minor updates merge under the org policy; majors keep the review gate.

@kiro-systemf kiro-systemf Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed at eabdafa against the organisation policy on systemfsoftware/.github main. The file follows the policy's shape: extends: .github, a repo-owned default queue rule naming this repository's checks, and the Dependabot patch/minor fast path spelled exactly as the policy's anchor does. Check names verified against this repository's real checks: checks / the gate, check / a publishable-package change needs an intent, and Same bytes on every host all exist; 13/13 pass at this head. Non-blocking: review-gate in the first or branch matches no check today — the policy's phase-2 spelling is @kiro-systemf/review-gate — so that branch is inert until phase 2; the approval and Dependabot branches carry the gate meanwhile.

Read the revised .mergify.yml at eabdafa against the org policy now on systemfsoftware/.github main, confirmed every check name but review-gate exists on this repository's PRs, and that 13/13 checks pass at the head.

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