← Back to issue list

fix: crash on certain YAML schema errors (#5696)

View original Github issue

Metadata

Project
snapcraft
Number
#5728
Type
pull request
State
merged
Author
steinbro
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #5728, fixing crashes caused by certain YAML schema errors and resolving #5696 in commit 3f076004f.

Resolution: closed resolved

Reason: PR #5728 was approved with all CI checks passing and merged as commit 3f076004f. Its description explicitly states that it fixes #5696.

Evaluated by: mr-cal-bot

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
stealth/space-bunny-alpha — closed resolved Merged PR #5728, fixing crashes caused by certain YAML schema errors and resolving #5696 in commit 3f076004f.
stealth/space-bunny-alpha — closed resolved Merged PR #5728 via commit 3f076004f, fixing crashes on certain YAML schema errors and resolving issue #5696.
stealth/space-bunny-alpha — closed resolved Merged the YAML schema-error crash fix in canonical/snapcraft#5728 via commit 3f076004f, resolving issue #5696.
stealth/space-bunny-alpha — closed resolved Merged the YAML schema-error crash fix in #5728 (commit 3f076004f), closing #5696.
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.

Update history

No update history recorded yet.

Related work

Related issues

Issue Project State Summary Similarity
#65 fix: correctly represent yaml dumped multiline strings craft-application merged Merged PR #65 added a YAML representer so multiline strings dump correctly; PR commit 85bb6a0 landed in main as 51346d9b.
72%