fix: provide more info in YAML errors
Metadata
Current evaluation
Merged to fix disappearing YAML error information, resolving issue #427. Approved by two reviewers, passed CI checks, and modified three files (+27/-2 lines) to restore detailed error reporting.
Suggested action: —
No scores available.
Issue body
- [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`?
-----
Fixes the information disappearing from #427. Thanks @mattculler for pointing this out.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged to fix disappearing YAML error information, resolving issue #427. Approved by two reviewers, passed CI checks, and modified three files (+27/-2 lines) to restore detailed error reporting. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix for YAML error messages losing critical information, resolving issue #427. The change restores full error details and improves debugging clarity. |
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. | |
| #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. | |
| #5849 docs: fix YAML reference types | snapcraft | merged | Merged documentation fix correcting YAML reference types. Approved by two reviewers, passed all CI checks, and updated one file with six additions and two deletions. | |
| #466 docs: fix yaml in examples | craft-parts | merged | Merged a documentation fix correcting YAML syntax in example parts.yaml by replacing underscores with hyphens. Approved by reviewer, passed CI, and changed two lines in one file. | |
| #816 Report the proper line number on bad yaml errors | snapcraft | merged | Merged to resolve Launchpad bug #1621166 by correcting YAML error reporting to display the accurate line number. Approved by one reviewer, the change updates four files with 26 additions and 2 deletions. | |
| #1004 fix: unhelpful error message when `base` is missing from YAML | rockcraft | merged | Merged a fix that improves the error message when the base field is missing from YAML, resolving issue 997. Approved by two reviewers, the PR passed CI checks aside from unrelated known failures, and was successfully integrated. | |
| #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. | |
| #2475 docs: fix incorrect indentation in YAML snippets | charmcraft | merged | Merged documentation fix for incorrectly indented YAML snippets caused by sphinx_toolbox. Replaces affected code-block directives with code directives to bypass the rendering bug. Changes span four files and were approved despite minor CI failures. | |
| #5483 fix: improve error presentation for yaml errors | snapcraft | merged | Merged a fix that prevents YAML errors from displaying as tracebacks when raised before the dispatcher initializes. The change resolves a craft-application 5 rebase regression and includes newly added tests. |