yaml_utils: allow unicode while encoding (#2414)
Metadata
Current evaluation
Merged pull request enabling unicode encoding in yaml_utils. The change passed all tests, achieved 100% diff coverage, and resolves Launchpad bug #1804258.
Suggested action: —
No scores available.
Issue body
LP: #1804258
Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>
- [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)?
- [ ] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)?
- [ ] Have you successfully run `./runtests.sh static`?
- [ ] Have you successfully run `./runtests.sh tests/unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged pull request enabling unicode encoding in yaml_utils. The change passed all tests, achieved 100% diff coverage, and resolves Launchpad bug #1804258. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged to enable Unicode support in yaml_utils encoding, resolving Launchpad bug #1804258. Codecov reported 100% diff coverage. A pending Rust error was noted for future fixes. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged PR #2414 to enable unicode encoding in yaml_utils, resolving Launchpad bug #1804258. The commit achieved 100% diff coverage. A maintainer noted a separate Rust error requires pending fixes. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2414 yaml_utils: allow unicode while encoding | snapcraft | merged | Merged to allow unicode encoding in yaml_utils, resolving Launchpad bug #1804258. The change modifies 4 files (+79/-1 lines), received approval, passed CI checks, and achieved 100% diff coverage. | |
| #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. | |
| #270 fix(yaml): allow use unicode in yaml | craft-application | merged | Merged a fix preventing unnecessary escaping of non-ASCII characters in YAML data and descriptions. Approved by two reviewers. The change modifies two files, adding 11 lines to properly handle unicode input. |