fix(yaml): wrap error message for YAML errors
Metadata
Current evaluation
Merged PR #427 (df906c31), wrapping YAML parsing failures in clearer YamlError diagnostics and fixing #407.
Resolution: closed resolved
Reason: PR #427 merged after approval by two reviewers. Commit df906c31 converts PyYAML failures into wrapped YamlError messages and resolves issue #407.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [ ] Have you followed the guidelines for contributing?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `tox`?
-----
Fixes #407
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #427 (df906c31), wrapping YAML parsing failures in clearer YamlError diagnostics and fixing #407. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in #427 (df906c31), fixing #407 by wrapping YAML error messages to improve invalid-file diagnostics. Approved by two reviewers with no unresolved comments. | |
| qwen/qwen3.6-35b-a3b | — | — | 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. |
Update history
No update history recorded yet.
Related work
-
Likely Fixed By:
canonical/craft-application#407
(confidence 100%)
Invalid project YAML diagnostics issue fixed by this PR and commit df906c31.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #433 fix: provide more info in YAML errors | craft-application | merged | PR #433 by maintainer lengau merged (commits 365cf2f5/de48adc3), restoring YAML error detail lost in #427: YamlError.from_yaml_error now sets details=str(error) and appends error.problem to the brief. |