← Back to issue list

fix(yaml): allow use unicode in yaml

View original Github issue

Metadata

Project
craft-application
Number
#270
Type
pull request
State
merged
Author
syu-w
Labels
—
Created
Updated
Closed

Current evaluation

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.

Suggested action: —

No scores available.

Issue body

When non-ASCII in the data, the entire text is escaped, especially in description.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b — — 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.
qwen3.6-35b-a3b-mtp-q6 — — Merged a fix allowing Unicode and non-ASCII characters in YAML data and descriptions without escaping the entire text. Resolves incorrect escaping behavior for non-ASCII content.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#427 fix(yaml): wrap error message for YAML errors craft-application merged Merged a fix wrapping YAML error messages, resolving issue #407. Approved by two reviewers, passed CI checks, and updated four files with 78 additions and 7 deletions.
76%
#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.
76%
#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.
75%
#2416 yaml_utils: allow unicode while encoding (#2414) snapcraft merged Merged pull request enabling unicode encoding in yaml_utils. The change passed all tests, achieved 100% diff coverage, and resolves Launchpad bug #1804258.
72%