← Back to issue list

improve "not a valid charm name" error

View original Github issue

Metadata

Project
charmcraft
Number
#734
Type
pull request
State
merged
Author
delgod
Labels
—
Created
Updated
Closed

Current evaluation

Merged in charmcraft#734 (commit 1ac7a8c0; merge fb50a098), clarifying the required lowercase-letter, alphanumeric-and-hyphen charm naming pattern.

Resolution: closed resolved

Reason: PR #734 was approved and merged into main as fb50a098. Its change, committed as 1ac7a8c0, expanded the invalid charm-name error with the accepted naming pattern.

Evaluated by: mr-cal-bot

No scores available.

Issue body

No description provided.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged in charmcraft#734 (commit 1ac7a8c0; merge fb50a098), clarifying the required lowercase-letter, alphanumeric-and-hyphen charm naming pattern.
stealth/space-bunny-alpha — closed resolved Merged charmcraft PR #711 to clarify the invalid charm-name error and explain the required naming pattern.
qwen/qwen3.6-35b-a3b — — Merged a one-line change to improve the error message for invalid charm names. Approved by a reviewer and merged on March 30, 2022.
qwen3.6-35b-a3b-mtp-q6 — — Merged improvement to the invalid charm name error message. External contributor delgod submitted the change, approved and merged by maintainer sergiusens.

Update history

No update history recorded yet.

Related work

  • Related To: charmcraft#711 (confidence 80%)

    An earlier same-titled change was merged as 51565d84 with commit 8beaa10a.

Related issues

Issue Project State Summary Similarity
#711 improve "not a valid charm name" error charmcraft merged Merged PR #711 (51565d84; 8beaa10a) to explain the required charm-name pattern in validation errors.
86%
#215 fix(charm): accept redundant multi-base charms craft-platforms merged Merged PR #215 as commit 4879309, allowing redundant multi-base charms when they produce equivalent valid build plans and addressing canonical/charmcraft#2620.
71%
#1974 Check `charmcraft.yaml` for charm name charmcraft merged Merged PR #1974 (commit beb5c145), fixing #1973: list-lib now reads the charm name from charmcraft.yaml instead of requiring metadata.yaml.
70%