fix: crash on certain YAML schema errors (#5696)
Metadata
Current evaluation
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.
Suggested action: —
No scores available.
Issue body
- [ ] Have you followed the [guidelines for contributing](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md)?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `make lint`?
- [ ] Have you successfully run `make test`?
---
Fixes #5696.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix for crashes triggered by specific YAML schema errors, resolving issue #5696. The changes were integrated after passing lint and test suites. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix preventing crashes caused by specific YAML schema errors, resolving issue #5696. Authored by maintainer steinbro, the change was successfully integrated without additional discussion. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #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. | |
| #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. | |
| #144 feat: dump_yaml utility | craft-application | merged | Merged the dump_yaml utility feature, closing issue #135. Approved by two reviewers, passed CI checks, and improved test coverage to 99.02%. Modified four files with 102 additions and 30 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. |