Skip to content

ci: exempt mergify-ci-bot from human review for autobump PRs#22

Merged
mergify[bot] merged 1 commit into
mainfrom
devs/kozlek/mrgfy-7687-exempt-mergify-ci-bot
Jun 17, 2026
Merged

ci: exempt mergify-ci-bot from human review for autobump PRs#22
mergify[bot] merged 1 commit into
mainfrom
devs/kozlek/mrgfy-7687-exempt-mergify-ci-bot

Conversation

@kozlek

@kozlek kozlek commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator

The new ci-bot homebrew-tap-sync.yml workflow opens mergify-cli
formula bump PRs into this repo. Those PRs are authored by the
mergify-ci-bot GitHub user (a plain user account, no [bot] suffix —
they're pushed with its PAT), so today they would be blocked on the
2-approval rule in 👀 Review Requirements.

This mirrors the existing dependabot[bot] exemption:

  • automated updates queue_rule — the single author = dependabot[bot]
    condition becomes an or: of dependabot[bot] and mergify-ci-bot, so
    the bump PR auto-queues once CI is green.
  • 👀 Review Requirements merge_protection — adds author != mergify-ci-bot
    alongside the existing author != dependabot[bot], so the 2-approval
    requirement does not apply to the bot's PRs.

Safe by construction: CI downloads and tests all four binaries, so a wrong
url/sha turns CI red and the PR never queues — no human approval is being
traded away for correctness.

Related to MRGFY-7687

🤖 Generated with Claude Code

The ci-bot homebrew-tap-sync workflow opens mergify-cli formula bump PRs
authored by the mergify-ci-bot user. Treat it like dependabot[bot]: add
it to the `automated updates` queue conditions and exclude it from the
`👀 Review Requirements` protection, so the bump PR auto-queues on
all-greens with no human approval. Safe by construction — CI downloads
and tests all four binaries, so a wrong url/sha turns CI red and the PR
never queues.

Related to MRGFY-7687

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Change-Id: If1c23685fcf46b333162e01b865df605192444c1
Copilot AI review requested due to automatic review settings June 17, 2026 06:48
@mergify mergify Bot deployed to Mergify Merge Protections June 17, 2026 06:48 Active
@mergify

mergify Bot commented Jun 17, 2026

Copy link
Copy Markdown

Merge Protections

🟢 All 6 merge protections satisfied — ready to merge.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates the repository’s Mergify configuration to treat PRs authored by mergify-ci-bot the same way as Dependabot PRs, allowing fully-CI-validated autobump PRs to merge without the human 2-approval requirement.

Changes:

  • Expands the automated updates queue rule to match PRs authored by either dependabot[bot] or mergify-ci-bot.
  • Updates the 👀 Review Requirements merge protection so the 2-approval requirement does not apply to PRs authored by mergify-ci-bot (mirroring the existing Dependabot exemption).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@kozlek kozlek requested a review from a team June 17, 2026 06:50
@mergify

mergify Bot commented Jun 17, 2026

Copy link
Copy Markdown

Queued — the merge queue status continues in this comment ↓.

@mergify

mergify Bot commented Jun 17, 2026

Copy link
Copy Markdown

Merge Queue Status

  • Entered queue2026-06-17 07:12 UTC · Rule: default
  • Checks skipped · PR is already up-to-date
  • Merged2026-06-17 07:13 UTC · at 962e9fe5716d4777b244b4b61e2a96beee80c9c0 · squash

This pull request spent 18 seconds in the queue, including 4 seconds running CI.

Required conditions to merge

@mergify mergify Bot added the queued label Jun 17, 2026
@mergify mergify Bot merged commit 5d83115 into main Jun 17, 2026
9 checks passed
@mergify mergify Bot removed the queued label Jun 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants