Skip to content

chore(mergify): update configuration#4306

Merged
greg-in-a-box merged 1 commit into
box:masterfrom
greg-in-a-box:mergify-con
Sep 29, 2025
Merged

chore(mergify): update configuration#4306
greg-in-a-box merged 1 commit into
box:masterfrom
greg-in-a-box:mergify-con

Conversation

@greg-in-a-box
Copy link
Copy Markdown
Contributor

@greg-in-a-box greg-in-a-box commented Sep 29, 2025

Summary by CodeRabbit

  • Chores
    • Updated automation rules to exclude actions on bot-authored pull requests, preventing unintended triggers.
    • Applied the exclusion across PR checks and both automatic merge queues for more predictable behavior.
    • Reduces noise, avoids circular automation, and improves reliability of merge processing.
    • No user-facing features or APIs changed.

@greg-in-a-box greg-in-a-box requested a review from a team as a code owner September 29, 2025 17:28
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Sep 29, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

Adds author != mergify[bot] conditions in .mergify.yml across pull_request_rules and two queue_rules, altering rule evaluation and queue admission to exclude actions on PRs authored by mergify[bot]. No code or API changes.

Changes

Cohort / File(s) Summary
Mergify rules
\.mergify.yml
Inserted author != mergify[bot] in relevant pull_request_rules and in queue_rules for “Automatic boxmoji merge” and “Automatic strict merge”.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor Dev as Developer/Bot
  participant GH as GitHub PR
  participant M as Mergify
  participant Q as Merge Queue

  Dev->>GH: Open/Update PR
  GH-->>M: Trigger rule evaluation
  M->>M: Check conditions (includes author != mergify[bot])
  alt Author is not mergify[bot]
    M->>GH: Apply matching pull_request_rules
    M->>Q: Enqueue PR (boxmoji/strict) if conditions pass
    Q-->>M: Queue status updates
  else Author is mergify[bot]
    M-->>GH: Skip actions and queueing for this PR
  end

  note over M,Q: New/changed condition gate: exclude PRs by mergify[bot]
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested labels

ready-to-merge

Suggested reviewers

  • tjuanitas
  • jpan-box

Poem

Mergify’s got rules, fool, listen to me!
Bots step back—no auto-merge spree.
Authors checked clean, queues stay tight,
Boxmoji, strict—merge only when right.
Keep it legit, don’t be no bot ruse—
I pity the fool who misuses these cues!

✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 818a45b and 5c05129.

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

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 and usage tips.

@greg-in-a-box greg-in-a-box merged commit 835bf3a into box:master Sep 29, 2025
4 checks passed
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