fix: adjust max summary length
Metadata
Current evaluation
Merged PR #1597 in commit ac159e49, raising the summary limit from 78 to 200 characters and documenting the recommendation to keep summaries shorter.
Resolution: closed resolved
Reason: PR #1597 was approved and merged as commit ac159e49. It raised the maximum summary length to 200 characters and linked #1598 for a future warning about summaries longer than 78 characters.
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 PR #1597 in commit ac159e49, raising the summary limit from 78 to 200 characters and documenting the recommendation to keep summaries shorter. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged as commit ac159e49: PR #1597 raised the charm summary field max length from craft-application's 78 to 200 characters in charmcraft/models/project.py, with a docstring/comment recommending shorter summaries. Follow-up #1598 (78-char lint warning) still open. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in ac159e49: raised the summary limit from 78 to 200 characters, documented the recommendation to keep summaries shorter, and linked #1598 to add a future warning. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #1597 as ac159e49, raising the input summary limit from 78 to 200 characters and documenting the planned linter in #1598. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged after two approvals. Adjusts max summary length, updates the docstring to recommend significantly shorter summaries, adds an explanatory comment, and links to issue #1598 to encourage shorter fields. Changes span one file with 16 additions and 4 deletions. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged after updating the docstring to recommend summaries significantly shorter than the 78-character limit. Added explanatory comments and linked to issue #1598 to guide users toward shorter summary fields. |
Update history
No update history recorded yet.
Related work
-
Related To:
canonical/charmcraft#1598
(confidence 99%)
Direct follow-up for warning when summaries exceed 78 characters; it remains open and does not supersede #1597.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1759 fix(metadata): allow long summaries in output | charmcraft | merged | Merged PR #1759 (commit 83c16214) allowed metadata output summaries longer than 78 characters while retaining the 200-character input limit, fixing #1758. | |
| #1440 summary field should be allowed more than 78 characters | charmcraft | closed | Closed as a duplicate of #1568, where the summary validation limit was raised from 78 to 200 characters; proposed PR #1585 was closed unmerged. | |
| #1758 Max summary length is limited in metadata output | charmcraft | closed | Removed the 78-character metadata-output limit by merging PR #1759 (commit 83c16214), while preserving the 200-character input limit. |