← Back to issue list

feat: dump_yaml utility

View original Github issue

Metadata

Project
craft-application
Number
#144
Type
pull request
State
merged
Author
lengau
Labels
Created
Updated
Closed

Current evaluation

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.

Suggested action:

No scores available.

Issue body

Closes #135 - [x] Have you followed the guidelines for contributing? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] Have you successfully run `tox`? -----

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b 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.
qwen3.6-35b-a3b-mtp-q6 Merged to introduce a dump_yaml utility function, closing issue #135. The update passed all tests, maintained 99%+ code coverage, and was approved by a maintainer.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#5728 fix: crash on certain YAML schema errors (#5696) snapcraft merged Merged to fix a crash on certain YAML schema errors, closing issue #5696. Approved by two reviewers and passing all CI checks, the change modifies two files with a +52/-1 diff.
72%
#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.
72%
#65 fix: correctly represent yaml dumped multiline strings craft-application merged 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.
71%