← Back to issue list

fix: improve error presentation for yaml errors

View original Github issue

Metadata

Project
snapcraft
Number
#5483
Type
pull request
State
merged
Author
mr-cal
Labels
PR: Squash
Created
Updated
Closed

Current evaluation

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.

Suggested action: —

No scores available.

Issue body

- [x] Have you followed the [guidelines for contributing](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md)? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] Have you successfully run `make lint`? - [x] Have you successfully run `make test`? --- This fixes a regression related to rebasing onto craft-application 5. Yaml-related errors were getting presented as tracebacks because they were raised before the dispatcher was created.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b — — 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.
qwen3.6-35b-a3b-mtp-q6 — — Merged a fix for a regression where YAML errors displayed as tracebacks before the dispatcher was created. Improved error presentation and added tests per reviewer feedback. Squash merged.
qwen3.6-35b-a3b-mtp-q6 — — Merged a fix addressing a regression where YAML errors displayed as tracebacks due to early exception handling before dispatcher initialization. Includes added tests per reviewer request.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#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.
72%
#433 fix: provide more info in YAML errors craft-application merged 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.
70%