Skip to content

Update merge queue configuration#13

Merged
w0rxbend merged 1 commit into
mainfrom
mergify/w0rxbend/I30dfe2572d49291186c62c66eaa4a612479d3d51
Jul 6, 2026
Merged

Update merge queue configuration#13
w0rxbend merged 1 commit into
mainfrom
mergify/w0rxbend/I30dfe2572d49291186c62c66eaa4a612479d3d51

Conversation

@w0rxbend

@w0rxbend w0rxbend commented Jul 6, 2026

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • Chores
    • Updated merge automation settings to use a clearer label-removal format.
    • Added merge queue configuration and set parallel checks to improve merge handling.

Signed-off-by: Oleksandr B. <null>
@coderabbitai

coderabbitai Bot commented Jul 6, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

This PR modifies .mergify.yml, changing the label-removal action for the "Delete the merge-when-green label after merge" rule to explicit list syntax, and adds merge queue configuration via queue_rules and merge_queue.max_parallel_checks settings.

Changes

Mergify Configuration Update

Layer / File(s) Summary
Label removal and merge queue configuration
.mergify.yml
Rewrites the block-merge label removal to explicit list form and adds queue_rules: [] and merge_queue.max_parallel_checks: 2 settings.

Estimated code review effort: 1 (Trivial) | ~3 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title matches the main change: updating merge queue configuration in the Mergify config.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch mergify/w0rxbend/I30dfe2572d49291186c62c66eaa4a612479d3d51

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@mergify

mergify Bot commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

@w0rxbend
w0rxbend merged commit 0392303 into main Jul 6, 2026
6 of 7 checks passed

@coderabbitai coderabbitai Bot 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.

Actionable comments posted: 1

🤖 Prompt for all review comments with 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.

Inline comments:
In @.mergify.yml:
- Around line 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.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 8cc043a6-e13b-444b-9722-27fdbc082e8d

📥 Commits

Reviewing files that changed from the base of the PR and between 260cf55 and 2075d3b.

📒 Files selected for processing (1)
  • .mergify.yml

Comment thread .mergify.yml
Comment on lines +19 to +21
queue_rules: []
merge_queue:
max_parallel_checks: 2

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.

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.

1 participant