← Back to issue list

Two small details before integrating help messages to main.

View original Github issue

Metadata

Project
charmcraft
Number
#163
Type
pull request
State
merged
Author
facundobatista
Labels
Created
Updated
Closed

Current evaluation

Merged after two approvals. Added an argsparsing option to CommandError for immediate help display and a helper to check command arguments. Modified four files with minor adjustments to prepare for a larger message refactoring.

Suggested action:

No scores available.

Issue body

Specifically: - support "argsparsing" option in CommandError to just present the message to the user (this will be refactored heavily in the short future when we change how all messages are sent to the user, but we need it now to just show the help) - Added a small helper to check for options in the arguments (which will be more used in the next branch).

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged after two approvals. Added an argsparsing option to CommandError for immediate help display and a helper to check command arguments. Modified four files with minor adjustments to prepare for a larger message refactoring.
qwen3.6-35b-a3b-mtp-q6 Merged. Added an arsparsing option to CommandError for immediate help display and a helper function for checking argument options. These changes prepare the codebase for upcoming help message refactoring and integration.
qwen3.6-35b-a3b-mtp-q6 Merged preparatory changes for help message integration. Added an argsparsing option to CommandError for immediate user display and introduced a helper to validate command-line options for future branches.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#154 Improved short help messages. charmcraft merged Merged improvements to short help messages for commands and parameters, updating output per UX suggestions. Approved by two reviewers, passed CI, and modified eight files as part of a broader help overhaul.
74%
#153 Adjust help messages in all cases to UX indications charmcraft merged Merged changes adding a dedicated help command, disabling automatic ArgumentParser help handling, and improving usage and error messages. Tests moved to test_help.py. Approved by two reviewers with 382 additions and 175 deletions across three files.
74%
#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.
73%