Skip to content

docs(merge-queue): direct merge never applies to a queue running in-place checks - #13035

Merged
mergify[bot] merged 1 commit into
Mergifyio:mainfrom
mergify-ci-bot:docs-agent/health-audit-merge-queue-3--mq3-direct-merge-excludes-in-place-checks
Sep 28, 2026
Merged

mergify[bot] merged 1 commit into
Mergifyio:mainfrom
mergify-ci-bot:docs-agent/health-audit-merge-queue-3--mq3-direct-merge-excludes-in-place-checks

Conversation

@mergify-ci-bot

Copy link
Copy Markdown
Contributor

The page listed three conditions a pull request has to meet before the queue can
merge it without a queue CI run, and left out the one that decides it for a whole
class of repositories: the optimization is the decision to skip creating a batch
pull request, and a queue configured for in-place checks never reaches that
decision at all. Such a queue tests the pull request on its own branch, so there
is no batch pull request and no extra CI run for Direct Merge to remove.

Without that line a reader on a serial, single-check queue could read the
"already up to date" case as applying to them, conclude Direct Merge was broken
when the queue behaved no differently, and go looking for a configuration
mistake that does not exist.

Evidence: the batch-checking path that asks whether the batch pull request can be
skipped is only entered when the queue builds one; the in-place path is a
separate entry point that never consults it.

@mergify
mergify Bot deployed to Mergify Merge Protections September 28, 2026 11:09 Active
@mergify

mergify Bot commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor

Merge Protections

🟢 All 5 merge protections satisfied — ready to merge.

Show 5 satisfied protections

🟢 👀 Review Requirements

  • any of:
    • #approved-reviews-by >= 1
    • author = dependabot[bot]
    • author = renovate[bot]
    • all of:
      • -head ~= ^docs-agent/
      • author = mergify-ci-bot

🟢 Enforce conventional commit

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|internal|docs|style|refactor|perf|test|build|ci|chore|revert|ui)(?:\(.+\))?!?:

🟢 🔎 Reviews

  • #changes-requested-reviews-by = 0
  • #review-requested = 0
  • #review-threads-unresolved = 0

🟢 📕 PR description

  • body ~= (?ms:.{48,})

🟢 🚦 Auto-queue

When all merge protections are satisfied, this pull request will be queued automatically.

@mergify
mergify Bot requested a review from a team September 28, 2026 11:11
@mergify

mergify Bot commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor

Merge Queue Status

This pull request spent 3 minutes 12 seconds in the queue, including 2 minutes 38 seconds running CI.

Required conditions to merge

@mergify mergify Bot added the queued label Sep 28, 2026
@mergify
mergify Bot merged commit 3d16334 into Mergifyio:main Sep 28, 2026
12 checks passed
@mergify mergify Bot removed the queued label Sep 28, 2026

This branch was successfully deployed

1 active deployment
Mergify Merge Protections — 055b1217 Deployed Sep 28, 2026 by mergify[bot]
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.

2 participants