fix: correctly represent yaml dumped multiline strings
Metadata
Current evaluation
Merged into main after passing CI and receiving two approvals. Adds a YAML representer to correctly dump multiline strings, aligning with other craft applications. Achieved 100% diff coverage.
Suggested action: —
No scores available.
Issue body
Add a representer as we do for other craft applications to correctly dump multi line strings.
- [ ] Have you followed the guidelines for contributing?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `tox`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged into main after passing CI and receiving two approvals. Adds a YAML representer to correctly dump multiline strings, aligning with other craft applications. Achieved 100% diff coverage. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into main. Added a custom YAML representer to correctly dump multiline strings, aligning with other craft applications. Passed coverage checks with 100% diff coverage. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #3716 meta: dump unicode yaml data | snapcraft | merged | Merged update to YAML dumping that enables Unicode support, fixing multiline text formatting in snap.yaml. Approved by reviewer, passed CI checks, and caused no coverage changes. | |
| #68 fix: order the yaml output | craft-application | merged | Merged PR reordering YAML output to match Craft Applications standards. Fields now follow name, version, summary, description, base/platform, metadatalinks, and parts order. Approved by two reviewers, passed CI, and integrated into main. | |
| #5483 fix: improve error presentation for yaml errors | snapcraft | merged | Merged a fix that prevents YAML errors from displaying as tracebacks when raised before the dispatcher initializes. The change resolves a craft-application 5 rebase regression and includes newly added tests. | |
| #433 fix: provide more info in YAML errors | craft-application | merged | Merged to fix disappearing YAML error information, resolving issue #427. Approved by two reviewers, passed CI checks, and modified three files (+27/-2 lines) to restore detailed error reporting. | |
| #144 feat: dump_yaml utility | craft-application | merged | Merged the dump_yaml utility feature, closing issue #135. Approved by two reviewers, passed CI checks, and improved test coverage to 99.02%. Modified four files with 102 additions and 30 deletions. |