Skip to content

feat(queue): add mergify queue show command#1069

Merged
mergify[bot] merged 1 commit intomainfrom
devs/jd/feat/queue-show/I08236237da0cb543e420fdd3f6380cb67b100b83
Apr 8, 2026
Merged

feat(queue): add mergify queue show command#1069
mergify[bot] merged 1 commit intomainfrom
devs/jd/feat/queue-show/I08236237da0cb543e420fdd3f6380cb67b100b83

Conversation

@jd
Copy link
Copy Markdown
Member

@jd jd commented Apr 7, 2026

Add a new CLI command to display the detailed state of a specific PR in
the merge queue — position, priority, CI checks with status icons,
and conditions evaluation tree.

Default output is compact: checks summary line + only blocking
conditions (top-level). Use --verbose/-v for full checks table and
conditions tree.

Closes MRGFY-6829

Co-Authored-By: Claude Opus 4.6 (1M context) noreply@anthropic.com

Copilot AI review requested due to automatic review settings April 7, 2026 18:57
@mergify mergify Bot had a problem deploying to Mergify Merge Protections April 7, 2026 18:57 Failure
@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Apr 7, 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 mergify Bot requested a review from a team April 7, 2026 18:59
Copy link
Copy Markdown

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

Adds a new mergify queue show CLI command to display a single PR’s merge-queue state, with compact vs --verbose output modes and optional JSON output.

Changes:

  • Implement queue show command with metadata, CI/checks rendering, and conditions evaluation output (compact/verbose).
  • Add typed API response shapes for the new “get queue PR” endpoint.
  • Add a dedicated test suite covering normal, verbose, JSON, and error cases.

Reviewed changes

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

File Description
mergify_cli/queue/cli.py Adds rendering helpers and the new queue show command implementation.
mergify_cli/queue/api.py Adds TypedDict models + get_queue_pull() API wrapper.
mergify_cli/tests/queue/test_show.py Adds tests for queue show output modes, JSON, and error handling.

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

Comment thread mergify_cli/queue/cli.py
Comment thread mergify_cli/tests/queue/test_show.py Outdated
@jd jd marked this pull request as ready for review April 7, 2026 19:04
@jd jd force-pushed the devs/jd/feat/queue-show/I08236237da0cb543e420fdd3f6380cb67b100b83 branch from 7d97f07 to 764b806 Compare April 7, 2026 19:11
@mergify mergify Bot had a problem deploying to Mergify Merge Protections April 7, 2026 19:11 Failure
@jd jd force-pushed the devs/jd/feat/queue-show/I08236237da0cb543e420fdd3f6380cb67b100b83 branch from 764b806 to ed550db Compare April 7, 2026 19:21
@mergify mergify Bot had a problem deploying to Mergify Merge Protections April 7, 2026 19:21 Failure
JulianMaurin
JulianMaurin previously approved these changes Apr 8, 2026
Comment thread mergify_cli/tests/queue/test_show.py
@mergify mergify Bot requested a review from a team April 8, 2026 07:18
Add a new CLI command to display the detailed state of a specific PR in
the merge queue — position, priority, CI checks with status icons,
and conditions evaluation tree.

Default output is compact: checks summary line + only blocking
conditions (top-level). Use --verbose/-v for full checks table and
conditions tree.

Closes MRGFY-6829

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Change-Id: I08236237da0cb543e420fdd3f6380cb67b100b83
Claude-Session-Id: 6201776f-44c8-4c2c-9496-ccf7b794ceae
@jd jd force-pushed the devs/jd/feat/queue-show/I08236237da0cb543e420fdd3f6380cb67b100b83 branch from ed550db to 9178e73 Compare April 8, 2026 07:49
@mergify mergify Bot dismissed JulianMaurin’s stale review April 8, 2026 07:49

Pull request has been modified.

@mergify mergify Bot deployed to Mergify Merge Protections April 8, 2026 07:49 Active
@mergify mergify Bot requested a review from a team April 8, 2026 07:56
@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Apr 8, 2026

Merge Queue Status

This pull request spent 2 minutes 27 seconds in the queue, including 2 minutes 6 seconds running CI.

Required conditions to merge

mergify Bot added a commit that referenced this pull request Apr 8, 2026
@mergify mergify Bot added the queued label Apr 8, 2026
@mergify mergify Bot merged commit bf6e186 into main Apr 8, 2026
19 checks passed
@mergify mergify Bot deleted the devs/jd/feat/queue-show/I08236237da0cb543e420fdd3f6380cb67b100b83 branch April 8, 2026 08:53
@mergify mergify Bot removed the queued label Apr 8, 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.

4 participants