fix(charm-libs): better error message on bad version type
Metadata
Current evaluation
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.
Suggested action: —
No scores available.
Issue body
Fixes #1954
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix improving error messages for invalid version types in charm-libs, resolving issue #1954. Authored by maintainer lengau. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged to resolve issue #1954. The change improves error messaging for invalid version types in the charm-libs module. Maintainer-authored and integrated into the codebase. |
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. | |
| #1954 error message is unhelpful when parsing charmcraft.yaml fails because `charm-libs[*].version` is an integer | charmcraft | closed | Fixed unhelpful AttributeError when charm-libs version is an integer. Merged in PR #1955, now returning a clear Pydantic validation error indicating the version field requires a string. | |
| #711 improve "not a valid charm name" error | charmcraft | merged | 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. | |
| #2034 fix(store): nicer error message when getting nonexistent libs | charmcraft | merged | Merged a fix that replaces a cryptic charmhub error with a clearer message when a charm library is missing. Approved by two reviewers, the change resolves issue #1754 and CRAFT-3353, modifying two files with 35 additions and 3 deletions. | |
| #1619 fix: use charmcraft package version | charmcraft | merged | Merged after approval from two reviewers. Implements a one-line fix to use the charmcraft package version, updating a single file to resolve the versioning configuration. |