Skip to content

fix(ci): match engine's merge queue PR title prefix#975

Merged
mergify[bot] merged 1 commit intomainfrom
devs/sileht/scope-bugged/Id5cf5a1187dddaa17f4a9c839db833a15444557d
Feb 19, 2026
Merged

fix(ci): match engine's merge queue PR title prefix#975
mergify[bot] merged 1 commit intomainfrom
devs/sileht/scope-bugged/Id5cf5a1187dddaa17f4a9c839db833a15444557d

Conversation

@sileht
Copy link
Member

@sileht sileht commented Feb 19, 2026

The engine generates MQ draft PR titles as "merge queue: ..." (with a
space) but the detector checked for "merge-queue: " (with a hyphen),
so the merge queue base SHA override never fired.

Fixes MRGFY-6270

Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com

The engine generates MQ draft PR titles as "merge queue: ..." (with a
space) but the detector checked for "merge-queue: " (with a hyphen),
so the merge queue base SHA override never fired.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Change-Id: Id5cf5a1187dddaa17f4a9c839db833a15444557d
Copilot AI review requested due to automatic review settings February 19, 2026 13:46
@mergify mergify bot deployed to Mergify Merge Protections February 19, 2026 13:46 Active
@mergify
Copy link
Contributor

mergify bot commented Feb 19, 2026

Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

🟢 🤖 Continuous Integration

Wonderful, this rule succeeded.
  • all of:
    • check-success=ci-gate

🟢 👀 Review Requirements

Wonderful, this rule succeeded.
  • any of:
    • #approved-reviews-by>=2
    • author = dependabot[bot]
    • author = mergify-ci-bot
    • author = renovate[bot]

🟢 Enforce conventional commit

Wonderful, this rule succeeded.

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

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

🟢 🔎 Reviews

Wonderful, this rule succeeded.
  • #changes-requested-reviews-by = 0
  • #review-requested = 0
  • #review-threads-unresolved = 0

🟢 📕 PR description

Wonderful, this rule succeeded.
  • body ~= (?ms:.{48,})

@mergify
Copy link
Contributor

mergify bot commented Feb 19, 2026

🧪 CI Insights

Here's what we observed from your CI run for 7abf1b3.

🟢 All jobs passed!

But CI Insights is watching 👀

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes a bug in the merge queue base SHA detection logic. The code was checking for PR titles starting with "merge-queue: " (with a hyphen), but the Mergify engine actually generates merge queue draft PR titles with "merge queue: " (with a space). This mismatch prevented the merge queue base SHA override from triggering.

Changes:

  • Updated the title prefix check in detector.py from "merge-queue: " to "merge queue: "
  • Updated the test case to use a more realistic merge queue title format that matches the actual engine output

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
mergify_cli/ci/git_refs/detector.py Fixed the title prefix check from "merge-queue: " to "merge queue: " to match engine output
mergify_cli/tests/ci/git_refs/test_git_refs_detector.py Updated test to use correct prefix and more realistic merge queue PR title

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mergify mergify bot requested a review from a team February 19, 2026 13:48
@mergify mergify bot merged commit 8bfc83a into main Feb 19, 2026
16 checks passed
@mergify mergify bot deleted the devs/sileht/scope-bugged/Id5cf5a1187dddaa17f4a9c839db833a15444557d branch February 19, 2026 13:56
@mergify
Copy link
Contributor

mergify bot commented Feb 19, 2026

Merge Queue Status

Rule: default


  • Entered queue2026-02-19 13:55 UTC
  • Checks passed · in-place
  • Merged2026-02-19 13:56 UTC · at 7abf1b32b93143e20bceaf21186551c48b5543f5

This pull request spent 40 seconds in the queue, including 6 seconds running CI.

Required conditions to merge

@mergify mergify bot added queued and removed queued labels Feb 19, 2026
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.

3 participants

Comments