fix: unhelpful error message when `base` is missing from YAML
Metadata
Current evaluation
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.
Suggested action: —
No scores available.
Issue body
Fixes #997
- [ ] Have you followed the guidelines for contributing?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `make lint && make test`?
---
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix improving the error message when the base field is missing from YAML. Resolves issue #997. Approved by maintainer @lengau, who noted unrelated test and lint failures were pre-existing. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged fix for unhelpful error message when base is missing from YAML, resolving issue #997. Approved by @lengau despite unrelated test and lint failures. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #5388 fix: don't fail when checking the base | snapcraft | merged | Merged a fix preventing snapcraft from failing early during base checks. Enables user-friendly error reporting for incorrect bases. Approved by two reviewers and merged with 10 additions and 30 deletions across three files. | |
| #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. | |
| #4342 refactor(parts): separate yaml parsing from base errors | snapcraft | merged | Merged into main after passing CI and receiving two approvals. The refactor separates YAML parsing from base errors to allow base validation without raising exceptions. No functional changes were made, and coverage increased by 0.01%. | |
| #4580 fix: do not propagate build-base to snap.yaml | snapcraft | merged | Merged a fix preventing build-base from propagating to snap.yaml. The pull request received two approvals, passed CI checks and code coverage, and was successfully integrated. | |
| #2626 schema: remove support for os when using bases | snapcraft | merged | Successfully merged pull request removing os type support from the schema when using bases, as base support supersedes it. Approved by reviewer, passed CI checks, and integrated. |