← Back to issue list

fix: don't suggest hidden commands

View original Github issue

Metadata

Project
craft-cli
Number
#302
Type
pull request
State
merged
Author
tigarmo
Labels
Created
Updated
Closed

Current evaluation

Merged to resolve issue #297 by preventing hidden command suggestions. Approved by two reviewers, passed CI checks, and updated three files with a +26/-1 line change.

Suggested action:

No scores available.

Issue body

Fixes #297 - [ ] Have you followed the guidelines for contributing? - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `tox`?

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged to resolve issue #297 by preventing hidden command suggestions. Approved by two reviewers, passed CI checks, and updated three files with a +26/-1 line change.
qwen3.6-35b-a3b-mtp-q6 Resolved issue #297 by preventing hidden commands from being suggested. The fix was merged into the codebase by maintainer tigarmo.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#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.
88%
#309 fix: hide positional arguments with no help craft-cli merged Merged, resolving issue #308. The change hides positional arguments without help text while preserving them in command syntax. Approved by two reviewers, passed CI checks, and updated three files (+102/-11 lines).
74%
#5375 test: fix command tests snapcraft merged Merged after updating test fixtures to resolve failing command unit tests. Approved by two reviewers, passed CI checks, and modified two files with 17 additions and 9 deletions.
71%
#72 Introduce the concept of hidden commands. craft-cli merged Merged after approval. Introduces hidden commands with 70 additions and 10 deletions across 8 files. A brief discussion on linting tool pinning occurred but did not block the merge.
71%
#365 fix: correct typo in default command warning craft-cli merged Merged a squashed pull request correcting a missing space typo in the default command warning. Approved by two reviewers with all CI checks passing.
70%