fix: merge command groups with the same name
Metadata
Current evaluation
Merged into main after three approvals. Consolidates identically named command groups to clean up help output, resolving issue #94. The change adds 41 lines and removes 1 across two files, achieving 100% test coverage.
Suggested action: —
No scores available.
Issue body
The use-case here is where the application provides commands in a logical "group" with the same name as of the default ones (a common example is an "init" command in the "Other" group).
Instead of having multiple command groups with the same name, which works fine but looks ugly when printing the help, merge command groups with the same name into a single one.
Fixes #94
- [ ] 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 into main after three approvals. Consolidates identically named command groups to clean up help output, resolving issue #94. The change adds 41 lines and removes 1 across two files, achieving 100% test coverage. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix consolidating duplicate command groups to clean up help output. Resolves #94. Successfully integrated into main with 100% test coverage. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #433 Converge all commands into one | snapcraft | merged | Merged a refactor that consolidates multiple commands into one, addressing Launchpad bugs #1567058 and #1567041. The change touches 35 files and removes over 1000 lines of code to eliminate redundant, interdependent logic. | |
| #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. | |
| #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. |