fix(charmcraft.yaml): relax summary string requirements
Metadata
Current evaluation
Merged to resolve issue #1568 by relaxing summary string validation in charmcraft.yaml. The update modifies two files with 175 additions and 13 deletions, closing the bug report without formal review.
Suggested action: —
No scores available.
Issue body
fixes #1568
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged to resolve issue #1568 by relaxing summary string validation in charmcraft.yaml. The update modifies two files with 175 additions and 13 deletions, closing the bug report without formal review. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix relaxing summary string validation in charmcraft.yaml, resolving issue #1568. The change permits broader string formats for the summary field while maintaining configuration compatibility. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged and closed issue #1568 by relaxing summary string requirements in charmcraft.yaml. Validation rules were updated to accept broader string formats. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2275 fix(charm): workaround to allow charmcraft-wide constraints | charmcraft | merged | Merged a workaround enabling charmcraft-wide constraints, resolving issue #2259 and CHARMCRAFT-614. Approved by two reviewers, the change modifies 15 files with 425 additions and passed CI checks prior to integration. | |
| #289 fix: fix issues found in common.mk with charmcraft | starbase | merged | Merged a fix for charmcraft issues in common.mk. Approved by two reviewers with zero unresolved comments. Changed two files, adding 11 and removing 7 lines. | |
| #471 config: add charm part to documentation (CRAFT-356) | charmcraft | merged | Merged changes adding charm configuration documentation to the configuration schema. Approved by two reviewers with zero unresolved comments. Updated two files with 15 additions and one deletion. Resolves CRAFT-356. | |
| #1622 fix: reorganise charmcraft.yaml preprocessing | charmcraft | merged | Merged following two approvals and passing CI. Reorganized charmcraft.yaml preprocessing to apply extensions on raw data, assign default parts to charms with bases, and enforce dictionary type for parts. Modified 5 files (+27/-14 lines). | |
| #188 Small text cleanups for README and charmcraft help | charmcraft | merged | Merged text cleanups for the README and charmcraft help documentation. Approved by two reviewers, the change updates 13 files to improve clarity while preserving test compatibility. | |
| #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. | |
| #1568 `charmcraft pack` fails with a false positive on the summary check | charmcraft | closed | The false positive validation error stemmed from a strict 78-character summary limit. The limit was increased to 200 characters to accommodate existing charms. The issue is closed, with follow-up PRs required to trim summaries on a few affected charms. | |
| #2801 docs: add Charmcraft 4.4 release notes | charmcraft | merged | Added Charmcraft 4.4 release notes. Merged after approval and passing CI checks. Updates four files (+188/-4 lines) to address CHARMCRAFT-711. | |
| #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. | |
| #46 New functionality from charmcraft (CRAFT-706). | craft-cli | merged | Merged new charmcraft functionality (CRAFT-706) following approval from two reviewers. The update adds approximately 2000 lines across 13 files, successfully integrating the feature set into the codebase. |