fix(remote-build): warn instead of error when using unsupported bases
Metadata
Current evaluation
Merged into main after approval and CI. Changed remote builds to warn instead of error on unsupported bases, resolving #4357. Diff coverage is 88.88% with a minor overall decrease.
Suggested action: —
No scores available.
Issue body
- [ ] 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 `make lint`?
- [ ] Have you successfully run `pytest tests/unit`?
-----
Fixes #4357
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged into main after approval and CI. Changed remote builds to warn instead of error on unsupported bases, resolving #4357. Diff coverage is 88.88% with a minor overall decrease. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into main. Remote builds now warn instead of erroring when unsupported bases are detected, resolving #4357. Approved by maintainer sergiusens despite a minor 0.01% test coverage decrease reported by Codecov. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into main, this change modifies remote build behavior to emit a warning instead of an error when unsupported bases are used, resolving issue #4357. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #4343 feat(remote-build): check base before executing | snapcraft | merged | Merged after reviewer approval and passing CI. Adds base validation before remote-build execution, raising errors for unsupported bases and enabling fallback for core20/core22. Resolves #4318. | |
| #2642 remote-build: detect early build errors | snapcraft | merged | Merged. Improved remote build error detection to distinguish between pending builders and actual failures like missing config or timeouts. Approved by reviewer, passed CI, and modified 3 files (+131/-47 lines). Minor import fix applied. | |
| #4480 fix(remote): return error when remote build failed | snapcraft | merged | Successfully merged fix for remote build failures that returns proper errors, resolving issue #4142. Approved by three reviewers with maintained code coverage across 13 files. | |
| #4432 fix(remote-build): improve logging and catch exceptions | snapcraft | merged | Merged pull request improving remote build logging and exception handling. Approved by reviewers, passed CI checks, and resolves issue #4399. | |
| #3182 build providers: improve warning for unknown base | snapcraft | merged | Merged after approval and passing CI checks. Improves the warning message for unknown bases in build providers, modifying one file with seven additions and four deletions. |