spread: update error when local snap is missing
Metadata
Current evaluation
Merged a one-line change to improve error reporting when a local snap is missing during spread updates. Approved by a reviewer, passed CI, and integrated into the repository.
Suggested action: —
No scores available.
Issue body
- [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] Have you successfully run `./runtests.sh static`?
- [x] Have you successfully run `./runtests.sh tests/unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged a one-line change to improve error reporting when a local snap is missing during spread updates. Approved by a reviewer, passed CI, and integrated into the repository. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged update to spread test framework error reporting when a local snap is missing. Passes static and unit tests. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged changes to the spread testing framework that improve error reporting when a local snap is missing. The update clarifies failure messages during spread execution. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1084 tests: check for submodules before running spread | charmcraft | merged | Merged changes verifying the snapd-testing-tools submodule before running Spread tests. Adds a clear error if missing. Approved by a reviewer, passed CI, and merged with minimal modifications. | |
| #2937 spread tests: do not attempt to remove snapd snap | snapcraft | merged | Merged a one-line fix to prevent spread tests from removing the snapd snap in LXD environments, resolving test failures caused by snapd's protected status. Approved by one reviewer with passing CI checks. | |
| #240 tests: check for submodules before running spread | rockcraft | merged | Merged. Adds a pre-check for the snapd-testing-tools submodule before running spread tests, providing a clear error if missing. Approved by two reviewers, passed CI, and merged successfully. | |
| #4105 tests: check for submodules before running spread | snapcraft | merged | Merged. Adds a pre-run check for the snapd-testing-tools submodule in spread tests to provide a clear error if missing. Approved by two reviewers, passed CI, and introduced no coverage changes. | |
| #1325 cli: proper error for failed snap command | snapcraft | merged | Merged: returns a proper error code when a snap command fails. Approved by one reviewer with no unresolved comments. Changes span two files (+15/-2). | |
| #4529 Spread update | snapcraft | merged | Merged after approval by two reviewers and passing CI checks. The update modifies spread configuration across three files, adding 36 and removing 40 lines. Code coverage tests passed. | |
| #712 fix(TestingService): provide a better error if spread.yaml is missing | craft-application | merged | Merged update that improves error messaging when spread.yaml is missing in the TestingService. Approved by two reviewers, passed CI checks, and includes a changelog entry. Resolves #703. | |
| #3158 spread: add snap watch | snapcraft | merged | Approved by one reviewer and merged. Adds snap watch to spread, modifying one file with 11 additions and 8 deletions. | |
| #5075 tests: fix validation sets spread test | snapcraft | merged | Merged fix for validation sets spread test switching to the hello-world snap and flipping the presence check instead of a specific revision. Approved by two reviewers with passing CI, resolving issue #5053. | |
| #3806 spread: add build packages to package repository tests | snapcraft | merged | Merged pull request adding spread tests to verify build packages install via snapcraft.yaml package repositories in destructive and build instance modes. Approved and merged with no coverage impact. |