improve "not a valid charm name" error
Metadata
Current evaluation
Merged an update to the "not a valid charm name" error message. The change clarifies the required naming pattern, eliminating the need to inspect source code. Approved by one reviewer and merged with a 5-line addition and 1-line deletion.
Suggested action: —
No scores available.
Issue body
Before that fix, it was needed to look into the source code to identify the correct pattern.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged an update to the "not a valid charm name" error message. The change clarifies the required naming pattern, eliminating the need to inspect source code. Approved by one reviewer and merged with a 5-line addition and 1-line deletion. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged an external contribution improving the invalid charm name error. The update provides a clearer validation message with the correct pattern, eliminating the need to inspect source code for formatting rules. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged an external contribution improving the invalid charm name error message. The update clarifies valid naming patterns, removing the need to inspect source code for validation rules. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #734 improve "not a valid charm name" error | charmcraft | merged | Merged a one-line change to improve the error message for invalid charm names. Approved by a reviewer and merged on March 30, 2022. | |
| #1974 Check `charmcraft.yaml` for charm name | charmcraft | merged | Merged changes to validate the charm name in charmcraft.yaml, resolving issue 1973. It passed CI, received two approvals, and was merged after the author corrected a CLA email mismatch via force push. | |
| #1955 fix(charm-libs): better error message on bad version type | charmcraft | merged | Merged to resolve issue #1954 by improving the error message for invalid version types in charm-libs. Approved by two reviewers with a minimal two-file change. |