← Back to issue list

fix: correctly represent yaml dumped multiline strings

View original Github issue

Metadata

Project
craft-application
Number
#65
Type
pull request
State
merged
Author
sergiusens
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #65 added a YAML representer so multiline strings dump correctly; PR commit 85bb6a0 landed in main as 51346d9b.

Resolution: closed resolved

Reason: PR #65 was approved with no unresolved review comments and merged into main. Commit 51346d9b implements the representer needed for correct multiline-string YAML output.

Evaluated by: mr-cal-bot

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
stealth/space-bunny-alpha — closed resolved Merged PR #65 added a YAML representer so multiline strings dump correctly; PR commit 85bb6a0 landed in main as 51346d9b.
stealth/space-bunny-alpha — closed resolved Merged PR #65 fixed YAML multiline-string dumping and landed as commit 51346d9b.
stealth/space-bunny-alpha — closed resolved Merged PR #65 (85bb6a0) added a YAML representer so multiline strings dump correctly, matching other Craft applications.
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.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#255 Properly represent multi-line yaml strings craft-application open Reported multi-line YAML formatting is already implemented in dump_yaml and covered by a regression test; commit 51346d9b is the resolving fix.
80%
#5728 fix: crash on certain YAML schema errors (#5696) snapcraft merged Merged PR #5728, fixing crashes caused by certain YAML schema errors and resolving #5696 in commit 3f076004f.
72%
#2033 LP: #1752344 snapcraft closed Fixed by the YAML-loading update merged in PR #2065 (commit 49f07a1ba).
72%
#466 docs: fix yaml in examples craft-parts merged Documentation example YAML was corrected in merged PR #466 (commit 5d34ba4c) by replacing invalid underscore-form keys with the required hyphenated form.
70%
#68 fix: order the yaml output craft-application merged Merged #68 via commit f5d94a77, ordering YAML output as identity/description fields, base/platform data, metadata links, and finally parts.
70%