Skip to content

feat(cli): support the git-style help subcommand#1682

Open
JulianMaurin wants to merge 3 commits into
mainfrom
devs/JulianMaurin/feat/rust-cli-excellence/support-git-style-help-subcommand--2f463a21
Open

feat(cli): support the git-style help subcommand#1682
JulianMaurin wants to merge 3 commits into
mainfrom
devs/JulianMaurin/feat/rust-cli-excellence/support-git-style-help-subcommand--2f463a21

Conversation

@JulianMaurin

@JulianMaurin JulianMaurin commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

mergify help and mergify help <command> failed with exit 2
("unrecognized subcommand 'help'") because disable_help_subcommand
was set, even though every other CLI offers the git-style help verb
alongside --help. Drop the flag so clap handles both forms.

Filter clap's synthetic help subcommand out of the published CLI
schema, the same way the --help/--version args are already
excluded — it's universal plumbing, not part of the reference.

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

Depends-On: #1681

@JulianMaurin JulianMaurin mentioned this pull request Jun 23, 2026
Open
@mergify mergify Bot had a problem deploying to Mergify Merge Protections June 23, 2026 18:29 Failure
This was referenced Jun 23, 2026
Merged
Merged
Merged
This was referenced Jun 23, 2026
Merged
Merged
Merged
Merged
@JulianMaurin

JulianMaurin commented Jun 23, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

This pull request is part of a Mergify stack:

# Pull Request Link
1 fix(ci): return a runtime error when scopes --write fails #1680
2 fix(stack): exit not-found when checkout finds no stack #1681
3 feat(cli): support the git-style help subcommand #1682 👈
4 docs(cli): correct stack reorder and scopes-send help text #1683
5 docs: document the post_classifying expect invariant and drop a stale Python reference #1691

@mergify

mergify Bot commented Jun 23, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Merge Protections

🔴 2 of 7 protections blocking · waiting on 👀 reviews

Protection Waiting on
🔴 👀 Review Requirements 👀 reviews
🔴 🔎 Reviews 👀 reviews
🟢 ⛓️ Depends-On Requirements —
🟢 🤖 Continuous Integration —
🟢 Enforce conventional commit —
🟢 📕 PR description —
🟢 🚦 Auto-queue —

🔴 👀 Review Requirements

Waiting for

  • #approved-reviews-by>=2
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
Show 5 satisfied protections

🟢 ⛓️ Depends-On Requirements

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

🟢 🤖 Continuous Integration

  • all of:
    • check-success=ci-gate

🟢 Enforce conventional commit

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

🟢 📕 PR description

  • body ~= (?ms:.{48,})

🟢 🚦 Auto-queue

When all merge protections are satisfied, this pull request will be queued automatically.

@JulianMaurin JulianMaurin force-pushed the devs/JulianMaurin/feat/rust-cli-excellence/exit-found-checkout-finds-stack--de52684a branch from 3011212 to c34f8a9 Compare June 25, 2026 07:49
@JulianMaurin JulianMaurin force-pushed the devs/JulianMaurin/feat/rust-cli-excellence/support-git-style-help-subcommand--2f463a21 branch from 9efb54a to 1e5b61c Compare June 25, 2026 07:49
@mergify mergify Bot had a problem deploying to Mergify Merge Protections June 25, 2026 07:49 Failure
@JulianMaurin

JulianMaurin commented Jun 25, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Revision history

# Type Changes Reason Date
1 initial 9efb54a 2026-06-25 07:49 UTC
2 rebase 9efb54a → 1e5b61c (rebase only) 2026-06-25 07:49 UTC
3 rebase 1e5b61c → 284a7d1 (rebase only) 2026-06-25 07:56 UTC
4 rebase 284a7d1 → f3d0eee (rebase only) 2026-06-25 08:51 UTC
5 rebase f3d0eee → a50f23c (rebase only) 2026-06-25 13:03 UTC

@mergify mergify Bot requested a review from a team June 25, 2026 07:55
@JulianMaurin JulianMaurin force-pushed the devs/JulianMaurin/feat/rust-cli-excellence/support-git-style-help-subcommand--2f463a21 branch from 1e5b61c to 284a7d1 Compare June 25, 2026 07:56
@JulianMaurin JulianMaurin force-pushed the devs/JulianMaurin/feat/rust-cli-excellence/exit-found-checkout-finds-stack--de52684a branch from c34f8a9 to 30e3f27 Compare June 25, 2026 07:56
@mergify mergify Bot had a problem deploying to Mergify Merge Protections June 25, 2026 07:56 Failure
@JulianMaurin JulianMaurin force-pushed the devs/JulianMaurin/feat/rust-cli-excellence/support-git-style-help-subcommand--2f463a21 branch from 284a7d1 to f3d0eee Compare June 25, 2026 08:51
@JulianMaurin JulianMaurin force-pushed the devs/JulianMaurin/feat/rust-cli-excellence/exit-found-checkout-finds-stack--de52684a branch from 30e3f27 to b706170 Compare June 25, 2026 08:51
@mergify mergify Bot had a problem deploying to Mergify Merge Protections June 25, 2026 08:51 Failure
@JulianMaurin JulianMaurin marked this pull request as ready for review June 25, 2026 09:24
JulianMaurin and others added 3 commits June 25, 2026 15:03
@JulianMaurin @claude
fix(ci): return a runtime error when scopes --write fails
33d7502 
`ci scopes --write <path>` mapped an unwritable output path to exit 8
(configuration error), but the config parsed fine — the failure is a
plain I/O error. Map it to the generic runtime code (exit 1) and keep
the OS error as the cause.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Change-Id: I1d4454775ae43a306ee2d4f5bb8fae2fec63f4a6
@JulianMaurin @claude
fix(stack): exit not-found when checkout finds no stack
1084872 
`stack checkout <name>` printed "No stacked pull requests found" but
exited 0, so a script couldn't tell a successful checkout from an
empty result. Return exit 3 (not-found) for the empty case, matching
`stack open`'s empty-stack handling.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Change-Id: Ide52684a19e6e16aff82888e95316d59f8e2767c
@JulianMaurin @claude
feat(cli): support the git-style help subcommand
a50f23c 
`mergify help` and `mergify help <command>` failed with exit 2
("unrecognized subcommand 'help'") because `disable_help_subcommand`
was set, even though every other CLI offers the git-style help verb
alongside `--help`. Drop the flag so clap handles both forms.

Filter clap's synthetic `help` subcommand out of the published CLI
schema, the same way the `--help`/`--version` args are already
excluded — it's universal plumbing, not part of the reference.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Change-Id: I2f463a21cb20c0901e93bd963783369d06bb2d41
@JulianMaurin JulianMaurin force-pushed the devs/JulianMaurin/feat/rust-cli-excellence/support-git-style-help-subcommand--2f463a21 branch from f3d0eee to a50f23c Compare June 25, 2026 13:03
@JulianMaurin JulianMaurin force-pushed the devs/JulianMaurin/feat/rust-cli-excellence/exit-found-checkout-finds-stack--de52684a branch from b706170 to 1084872 Compare June 25, 2026 13:03
@JulianMaurin JulianMaurin mentioned this pull request Jun 25, 2026
Open
@mergify mergify Bot had a problem deploying to Mergify Merge Protections June 25, 2026 13:03 Failure
Base automatically changed from devs/JulianMaurin/feat/rust-cli-excellence/exit-found-checkout-finds-stack--de52684a to main June 25, 2026 15:57
@mergify mergify Bot requested a review from a team June 25, 2026 16:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kozlek kozlek kozlek approved these changes

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@JulianMaurin @kozlek