← Back to issue list

fix(cli): enable per-command help queries

View original Github issue

Metadata

Project
craft-application
Number
#84
Type
pull request
State
merged
Author
lengau
Labels
Created
Updated
Closed

Current evaluation

Merged to main, enabling per-command help queries in the CLI and resolving issue #76. Approved by two reviewers, passed CI checks, and achieved 100% diff coverage across three files.

Suggested action:

No scores available.

Issue body

- [x] Have you followed the guidelines for contributing? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] Have you successfully run `tox`? ----- Fixes #76 This is further tested in Charmcraft's craft-application pack testing.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged to main, enabling per-command help queries in the CLI and resolving issue #76. Approved by two reviewers, passed CI checks, and achieved 100% diff coverage across three files.
qwen3.6-35b-a3b-mtp-q6 Merged into main, this change resolves issue #76 by enabling per-command help queries in the CLI. It passed all CI checks, including tox and CLA verification, and achieved 100% diff coverage prior to integration.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#1789 cli: improve the help command snapcraft merged Improved the CLI help command to match --help parity and expose hidden topics and plugin help. Merged after reviewer approval and passing CI, modifying three files with 62 additions and 11 deletions.
77%
#2429 cli: fix usage string in help command snapcraft merged Merged a one-line fix that corrected an incorrect usage string in the CLI help command. The change was approved, passed CI checks, and resolved the reported bug.
73%
#3682 cli: fix help command legacy fallback snapcraft merged Merged fix for the CLI help command legacy fallback. Handles craft-cli's ProvideHelpException to standardize help output. Approved, passed CI, and achieved 100% diff coverage before merging into the 7.0 branch.
72%
#95 fix: better help for invalid command args craft-application merged Merged into main after approval and CI checks. Resolves #89 by replacing the internal error for invalid command arguments with the standard help message, matching dispatcher behavior. Changes span 3 files with 100% diff coverage.
72%
#306 feat: remove code backticks in CLI help messages craft-cli merged Merged following approval by two reviewers and successful CI checks. The update removes code backticks from CLI help messages, closes #303. Changes affect two files with 90 additions and zero deletions.
72%
#452 fix: don't recommend hidden commands craft-cli merged Merged fix preventing hidden commands from appearing in the 'See also' section of command help text. Resolves issue #449. Approved by two reviewers with all CI checks passing. Modified three files.
72%
#196 feat: allow the application to choose the help order of commands craft-cli merged Merged to main, resolving issue #195. The feature allows applications to customize command help order. Approved by one reviewer, passed CI, and achieved 100% diff coverage. A reviewer requested a shorter commit message before merging.
71%
#1845 cli: implement help <command> snapcraft merged Implemented snapcraft help <command> CLI functionality. Merged after passing CI checks and receiving reviewer approval. Closes issue #1699.
70%
#1640 cli: add what, why, and how to fix to the common errors snapcraft merged Merged pull request adding what, why, and how to fix guidance to common CLI error messages. Approved by two reviewers, modified 15 files (+484/-155 lines), and passed CI after a temporary test failure was rerun.
70%