Skip to content

Update merge queue configuration #13

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
w0rxbend merged 1 commit into from
Jul 6, 2026
Merged

Update merge queue configuration #13

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 5 additions & 2 deletions .mergify.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,11 +8,14 @@ pull_request_rules:
actions:
merge:
method: squash

- name: Delete the merge-when-green label after merge
conditions:
- merged
- label=block-merge
actions:
label:
remove: [block-merge]
remove:
- block-merge
queue_rules: []
merge_queue:
max_parallel_checks: 2
Comment on lines +19 to +21

@coderabbitai coderabbitai Bot Jul 6, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
set -euo pipefail

# Inspect the target file with line numbers.
if [ -f .mergify.yml ]; then
  echo "== .mergify.yml =="
  cat -n .mergify.yml
else
  echo ".mergify.yml not found"
fi

# Find any queue-related Mergify config elsewhere.
echo
echo "== queue-related matches =="
rg -n --hidden --glob '.mergify.yml' --glob '.mergify.yaml' --glob '.github/**' 'queue_rules|merge_queue|actions:\s*$|queue:' .

# Show a compact file list for Mergify-related configs.
echo
echo "== mergify-related files =="
git ls-files | rg '(^|/)\.mergify\.ya?ml$|mergify'

Repository: w0rxbend/system-bootstrap

Length of output: 997

🏁 Script executed:

#!/bin/bash
set -euo pipefail

# If .mergify.yml is present, inspect surrounding context for the rule using merge vs queue.
python3 - <<'PY'
from pathlib import Path
p = Path('.mergify.yml')
if not p.exists():
    print("missing")
    raise SystemExit(0)
lines = p.read_text().splitlines()
for i, line in enumerate(lines, start=1):
    if 1 <= i <= len(lines):
        if 1 <= i <= len(lines):
            pass
for start in [1]:
    end = min(len(lines), 120)
    for i in range(start, end + 1):
        print(f"{i:4d}: {lines[i-1]}")
PY

Repository: w0rxbend/system-bootstrap

Length of output: 760

🏁 Script executed:

#!/bin/bash
set -euo pipefail

# Read Mergify docs references present in the repo, if any.
rg -n --hidden 'max_parallel_checks|queue_rules|merge_queue|queue action|merge queue' .

Repository: w0rxbend/system-bootstrap

Length of output: 271

merge_queue.max_parallel_checks is dead config here .mergify.yml:19-21 defines no queue rules, and the PR rule still uses merge: directly. Add a queue rule plus queue action if merge-queue behavior is intended; otherwise drop these lines.

🤖 Prompt for AI Agents 
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In @.mergify.yml around lines 19 - 21, The merge queue setting is unused because
there are no queue rules and the existing PR rule still merges directly. Update
the Mergify config by either adding a proper queue rule and switching the
relevant rule to use the queue action if merge-queue behavior is intended, or
remove the merge_queue.max_parallel_checks and queue_rules entries entirely.
Check the existing Mergify rule definitions around merge: and queue_rules to
keep the configuration consistent.

Loading