Repository navigation
docs(mergify): enqueue eligible PRs explicitly with @Mergifyio queue - #279
Conversation
AGENTS.md and docs/setup/mergify.md said every eligible PR is queued automatically. Observed on 2026-10-04: the last 14 merged PRs (#240-#266) and the open #274/#275 each carry an explicit "@Mergifyio queue" comment, so the documented automatic pickup is not what happens in practice. Keep the eligibility conditions, add the explicit step (gh pr checks <n>, then comment "@Mergifyio queue") and mark the cause as unresolved. No change to .mergify.yml, labels, account settings or any PR. No-Test: documentation-only correction; no runtime effect. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 70ad850a11
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| - **Einreihen ist ein eigener Schritt** (beobachtet 04.10.2026): Aufnahmefähige | ||
| PRs kamen nicht von selbst in die Queue. Die letzten 14 gemergten PRs | ||
| (#240–#266) und die offenen #274/#275 tragen je einen Kommentar | ||
| `@Mergifyio queue` (bzw. `requeue`) vom Koordinator. Ablauf: |
There was a problem hiding this comment.
Verify manual queueing before making it mandatory
The presence of queue comments proves only that coordinators posted them, not that eligible PRs would otherwise remain unqueued. This is especially important because the declared source of truth, .mergify.yml, still configures matching merge_protections_settings.auto_merge_conditions, which Mergify documents as automatically adding matching PRs to the queue. Confirm the behavior with an eligible PR left uncommented or with the Mergify account/dashboard state before documenting a mandatory human step; otherwise this introduces an unnecessary manual gate based on an unsupported causal claim.
AGENTS.md reference: AGENTS.md:L40-L41
Useful? React with 👍 / 👎.
|
Queued — the merge queue status continues in this comment ↓. |
|
@Mergifyio queue |
Merge Queue Status
This pull request spent 13 minutes 33 seconds in the queue, including 13 minutes 18 seconds running CI. Required conditions to merge
|
Die Anleitung behauptete, fertige PRs kämen von selbst in die Mergify-Warteschlange – in der Praxis hat der Koordinator jeden PR mit dem Kommentar
@Mergifyio queueeingereiht.AGENTS.mdunddocs/setup/mergify.mdbeschreiben diesen Schritt jetzt ausdrücklich; warum die automatische Aufnahme nicht greift, bleibt offen. Es ist reine Doku: an der Queue, an Labels, am Mergify-Konto und an PRs ändert sich nichts.Report
Package: documentation correction, Mergify explicit enqueue (not an M package, no spec file). Branch
claude/mergify-explicit-enqueue, head70ad850a11840c465e8a4d79a13fbd8843f927e6, branch point7ea73b2; current PR base4846c84(three main commits newer).What changed
AGENTS.md("Merging"): the PR conditions (non-draft, required checks green, no conflict, nodo-not-merge) are now described as eligibility, not as automatic queueing. Added the explicit step: whengh pr checks <n>is green, the coordinator comments@Mergifyio queue. The cause is marked unresolved (hypotheses only); Mergify still checks the queue run onmain.docs/setup/mergify.md: same correction in German ("Aufnahmefähig" + "Einreihen ist ein eigener Schritt", observation date 04.10.2026,gh pr comment <n> --body "@Mergifyio queue"), plus item 7 in "Warum steht mein PR nicht in der Queue?"..mergify.yml, branch protection, labels, Mergify account/dashboard or other PRs.Evidence (2026-10-04)
gh pr view <n> --json commentswith the filter@[Mm]ergify(io)? (queue|requeue). A checked set of 14 merged PRs (test(hygiene): accept a PLAN with zero in-progress packages #240, docs(plan): sync package status with merged PRs, keep E13 (PLAN-STATUS-03) #242, refactor(settings): extract UpdatesTab from SettingsView (ARCH-08d) #244, refactor(api): move goal routes into hq_routes (ARCH-10c) #246, refactor(settings): extract CategoriesPanel from SettingsView (ARCH-08e) #247, refactor(api): move task and control routes into hq_routes (ARCH-10d) #250, docs: state HQ proof boundaries for W3-08 and consumers #251, refactor(settings): extract ProfilesPanel (ARCH-08f) #252, Share the Windows replace primitive with DB restore (ARCH-03c) #256, docs: reconcile PLAN, STAND and KNOWN_ISSUES with live PR state #258, refactor(ipc): move project wrappers into ipc/projects.ts (ARCH-09a) #260, test: show drain unresolved ids and completion results in assertion message #262, refactor(ipc): move worker wrappers into ipc/workers.ts (ARCH-09b) #264, docs: sync PLAN, STAND and KI-30 to main cc95a57 #266) each has at least one match; fix(hq): ein 'teilweise:'-Paket in der Stand-Spalte nicht als fertig zählen (W1-17) #274 and docs(setup-15): correct provider, routing and script references in docs/setup #275 also have queue/requeue comments. Spot check test(hygiene): accept a PLAN with zero in-progress packages #240/docs: sync PLAN, STAND and KI-30 to main cc95a57 #266/docs(setup-15): correct provider, routing and script references in docs/setup #275: the comment@Mergifyio queueis byCuarroc(coordinator), followed by a Mergify bot reply (docs(setup-15): correct provider, routing and script references in docs/setup #275: no reply yet).npm run dev:start-check -- --jsonbefore the gates: exit 0 (3.9 GB free RAM, 0 cargo builds; provider limit not checked by the script, coordinator observed Claude 35 % for the week).bash scripts/ci/secret-scan.shexit 0, no leaks.bash scripts/ci/gates.sh lane prepushin this worktree on70ad850: exit 0, 427 s, all 11 gates green (rust-suite 191 s). A first attempt failedtypecheck(TS2688) because the new worktree had nonode_modules; fixed withnpm cifrom the lockfile, no repo change.git ls-remote origin refs/heads/claude/mergify-explicit-enqueue=70ad850a11840c465e8a4d79a13fbd8843f927e6.lane-plan.sh --classify:AGENTS.mdheavy (gates read it literally),docs/setup/mergify.mdlight, so the linux lane runs on this PR.Test trailer
No-Test: documentation-only correction; no runtime effect.Review disposition
Tier C (documentation without runtime effect): no external review; the gates are enough (AGENTS.md rule 5).
NICHT ABGEDECKT
🤖 Generated with Claude Code