improve "not a valid charm name" error
Metadata
Current evaluation
Merged PR #711 (51565d84; 8beaa10a) to explain the required charm-name pattern in validation errors.
Resolution: closed resolved
Reason: PR #711 was merged in commit 51565d84 with implementation commit 8beaa10a, clarifying that names must start with a lowercase letter and contain only alphanumeric characters and hyphens.
Evaluated by: mr-cal-bot
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 |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #711 (51565d84; 8beaa10a) to explain the required charm-name pattern in validation errors. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #734 (commit 1ac7a8c0), improving the invalid charm-name error to explain the required naming pattern without inspecting source code. | |
| 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. |
Update history
No update history recorded yet.
Related work
-
Related To:
charmcraft#734
(confidence 95%)
A later same-titled change made the error-text improvement again, merging via fb50a098 with commit 1ac7a8c0.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #734 improve "not a valid charm name" error | charmcraft | merged | Merged in charmcraft#734 (commit 1ac7a8c0; merge fb50a098), clarifying the required lowercase-letter, alphanumeric-and-hyphen charm naming pattern. |