Skip to content

feat: add inline CI, review, and merge status to mergify stack list#1117

Open
jd wants to merge 1 commit intodevs/jd/feat/pre-push-hook-intercept/I14bca20f08f8fcd5d265c86ebd5e1fd2eac688c6from
devs/jd/feat/pre-push-hook-intercept/I546622141e7fec536c9fa26c2f0a8a50cc59fa60
Open

feat: add inline CI, review, and merge status to mergify stack list#1117
jd wants to merge 1 commit intodevs/jd/feat/pre-push-hook-intercept/I14bca20f08f8fcd5d265c86ebd5e1fd2eac688c6from
devs/jd/feat/pre-push-hook-intercept/I546622141e7fec536c9fa26c2f0a8a50cc59fa60

Conversation

@jd
Copy link
Copy Markdown
Member

@jd jd commented Apr 10, 2026

Enhances mergify stack list to show CI status, review status, and
merge conflict state for each PR in the stack.

  • Default: summary line (CI: ✓ passing | Review: ✓ approved)
  • --verbose: detailed check names and reviewer names
  • --json: full structured data (ci_checks, reviews, mergeable)
  • Conflict indicator (✗ conflicting) when PR has merge conflicts

Fetches check-runs and reviews in parallel with bounded concurrency.
Callers like stack open skip the extra fetches via include_status=False.

Claude-Session-Id: 2d9f4a52-4a1d-4a50-a459-3b57847dec9f

Depends-On: #1116

@jd
Copy link
Copy Markdown
Member Author

jd commented Apr 10, 2026

This pull request is part of a stack:

  1. feat: add mergify stack sync CLI command (#1116)
  2. feat: add inline CI, review, and merge status to mergify stack list (#1117) 👈

@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Apr 10, 2026

Merge Protections

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

🔴 ⛓️ Depends-On Requirements

Waiting for:

This rule is failing.

Requirement based on the presence of Depends-On in the body of the pull request

🔴 👀 Review Requirements

Waiting for:

  • #approved-reviews-by>=2

or

  • author = dependabot[bot]

or

  • author = mergify-ci-bot

or

  • author = renovate[bot]
This rule is failing.
  • any of:
    • #approved-reviews-by>=2
    • author = dependabot[bot]
    • author = mergify-ci-bot
    • author = renovate[bot]

🔴 🔎 Reviews

Waiting for:

  • #review-requested = 0
This rule is failing.
  • #review-requested = 0
  • #changes-requested-reviews-by = 0
  • #review-threads-unresolved = 0

🟢 🤖 Continuous Integration

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

🟢 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)(?:\(.+\))?:

🟢 📕 PR description

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

@jd jd force-pushed the devs/jd/feat/pre-push-hook-intercept/I546622141e7fec536c9fa26c2f0a8a50cc59fa60 branch from 6e90bcc to 8f1be1e Compare April 10, 2026 09:13
@mergify mergify bot had a problem deploying to Mergify Merge Protections April 10, 2026 09:14 Failure
@jd jd marked this pull request as ready for review April 10, 2026 09:22
Copilot AI review requested due to automatic review settings April 10, 2026 09:22
@jd jd force-pushed the devs/jd/feat/pre-push-hook-intercept/I546622141e7fec536c9fa26c2f0a8a50cc59fa60 branch from 8f1be1e to 7a8d36d Compare April 10, 2026 09:24
@mergify mergify bot had a problem deploying to Mergify Merge Protections April 10, 2026 09:24 Failure
@mergify mergify bot requested a review from a team April 10, 2026 09:27
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

Enhances the mergify stack list command to surface per-PR CI status, review status, and merge-conflict (mergeable) state, with richer display options and JSON output suitable for scripting.

Changes:

  • Extend stack list entries with CI checks, reviews, and mergeable, and fetch those details via GitHub APIs with bounded concurrency.
  • Add --verbose/-v output to show individual check names and reviewer names; extend --json output with structured status fields.
  • Update stack open to skip the additional status fetches via include_status=False; update docs and tests accordingly.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
mergify_cli/stack/list.py Adds CI/review/mergeable fields, parallel status fetching, and verbose + formatted display output.
mergify_cli/stack/cli.py Wires new --verbose/-v option into stack list.
mergify_cli/stack/open.py Avoids extra CI/review API calls when listing is only used to open PRs.
mergify_cli/github_types.py Adds mergeable to the PR TypedDict.
mergify_cli/stack/skill.md Documents the enhanced stack list behavior and flags.
mergify_cli/tests/stack/test_list.py Adds/updates tests for JSON fields, summary output, and verbose output.
mergify_cli/tests/stack/test_open.py Updates PR payload mocks to include mergeable.

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

sileht
sileht previously approved these changes Apr 10, 2026
@mergify mergify bot requested a review from a team April 10, 2026 09:28
@jd jd force-pushed the devs/jd/feat/pre-push-hook-intercept/I546622141e7fec536c9fa26c2f0a8a50cc59fa60 branch from 7a8d36d to 3646ca8 Compare April 10, 2026 09:36
@jd jd force-pushed the devs/jd/feat/pre-push-hook-intercept/I14bca20f08f8fcd5d265c86ebd5e1fd2eac688c6 branch from 6456b6d to 9a98892 Compare April 10, 2026 09:36
@mergify mergify bot dismissed sileht’s stale review April 10, 2026 09:37

Pull request has been modified.

@mergify mergify bot had a problem deploying to Mergify Merge Protections April 10, 2026 09:37 Failure
sileht
sileht previously approved these changes Apr 10, 2026
@mergify mergify bot requested a review from a team April 10, 2026 13:45
@jd jd force-pushed the devs/jd/feat/pre-push-hook-intercept/I14bca20f08f8fcd5d265c86ebd5e1fd2eac688c6 branch from 9a98892 to 7dc6609 Compare April 10, 2026 13:55
@jd jd force-pushed the devs/jd/feat/pre-push-hook-intercept/I546622141e7fec536c9fa26c2f0a8a50cc59fa60 branch from 3646ca8 to 6fa8c18 Compare April 10, 2026 13:55
@mergify mergify bot dismissed sileht’s stale review April 10, 2026 13:56

Pull request has been modified.

@mergify mergify bot had a problem deploying to Mergify Merge Protections April 10, 2026 13:56 Failure
@jd jd force-pushed the devs/jd/feat/pre-push-hook-intercept/I546622141e7fec536c9fa26c2f0a8a50cc59fa60 branch from 6fa8c18 to 217c07b Compare April 11, 2026 07:17
@mergify mergify bot had a problem deploying to Mergify Merge Protections April 11, 2026 07:17 Failure
Enhances `mergify stack list` to show CI status, review status, and
merge conflict state for each PR in the stack.

- Default: summary line (CI: ✓ passing | Review: ✓ approved)
- --verbose: detailed check names and reviewer names
- --json: full structured data (ci_checks, reviews, mergeable)
- Conflict indicator (✗ conflicting) when PR has merge conflicts

Fetches check-runs and reviews in parallel with bounded concurrency.
Callers like `stack open` skip the extra fetches via include_status=False.

Change-Id: I546622141e7fec536c9fa26c2f0a8a50cc59fa60
Claude-Session-Id: 2d9f4a52-4a1d-4a50-a459-3b57847dec9f
@jd jd force-pushed the devs/jd/feat/pre-push-hook-intercept/I14bca20f08f8fcd5d265c86ebd5e1fd2eac688c6 branch from 515e148 to f0e7332 Compare April 11, 2026 07:42
@jd jd force-pushed the devs/jd/feat/pre-push-hook-intercept/I546622141e7fec536c9fa26c2f0a8a50cc59fa60 branch from 217c07b to 79efc36 Compare April 11, 2026 07:42
@mergify mergify bot had a problem deploying to Mergify Merge Protections April 11, 2026 07:42 Failure
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