← Back to issue list

fix(test): parse test command arguments

View original Github issue

Metadata

Project
charmcraft
Number
#1686
Type
pull request
State
merged
Author
cmatsuoka
Labels
—
Created
Updated
Closed

Current evaluation

Merged #1686 as commit 58d36668, making the test command parse arguments, support consistent double-dashed long options, and provide clearer help.

Resolution: closed resolved

Reason: PR #1686 was approved and merged as commit 58d36668. It replaced raw forwarding of test arguments to spread with argument parsing and improved help messages.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Let the test command parse its arguments instead of forwarding them to spread in raw form. This allows us to use consistent double-dashed long options, and have better help messages. Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged #1686 as commit 58d36668, making the test command parse arguments, support consistent double-dashed long options, and provide clearer help.
stealth/space-bunny-alpha — closed resolved Merged PR #1686 as commit 58d36668, making the test command parse its arguments and use consistent double-dashed options with improved help.
qwen/qwen3.6-35b-a3b — — Merged a fix updating the test command to parse arguments directly instead of forwarding them raw to spread. This enables consistent double-dashed long options and improved help messages. Approved by two reviewers, modifying five files with 55 additions and 32 deletions.
qwen3.6-35b-a3b-mtp-q6 — — Merged a fix that updates the test command to parse arguments directly instead of forwarding them raw to spread. This enables consistent double-dashed long options and improves help message formatting.

Update history

No update history recorded yet.

Related issues

No related issues found above the similarity threshold.