meta: split up package repository sanity checks
Metadata
Current evaluation
Merged. Splits package repository sanity checks into unique RuntimeErrors to aid developer debugging if schema validation fails. Approved after reviewer noted existing validation tools, with author clarifying the changes serve as a developer safety net.
Suggested action: —
No scores available.
Issue body
These sanity checks are supposed to be covered by the schema,
but in the event the schema fails, it's difficult to diagnose.
Provide a unique RuntimeError for each unexpected case.
Signed-off-by: Chris Patterson <chris.patterson@canonical.com>
- [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `./runtests.sh static`?
- [ ] Have you successfully run `./runtests.sh tests/unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged. Splits package repository sanity checks into unique RuntimeErrors to aid developer debugging if schema validation fails. Approved after reviewer noted existing validation tools, with author clarifying the changes serve as a developer safety net. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged changes splitting package repository sanity checks into distinct RuntimeErrors. Provides a developer safety net for schema validation failures, complementing existing validation-failure handling. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Splits package repository sanity checks into distinct RuntimeError exceptions to aid debugging when schema validation fails. Maintainer confirmed these serve as internal safety nets for code-level edge cases. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #3072 schema: minor tweaks/fixes for package-repositories | snapcraft | merged | Merged minor schema tweaks for package-repositories. Changes include grammar fixes, expanded test coverage for empty fields, corrected test assertions, and removed an unnecessary name validation. Approved by one reviewer and passed CI checks. |