refactor: subclass remote-build errors from CraftError
Metadata
Current evaluation
Merged craft-application#482 as be2750d8, making RemoteBuild errors CraftError subclasses, moving details into brief messages, and documenting release 4.2.4.
Resolution: closed resolved
Reason: PR #482 was merged as commit be2750d8, completing the CRAFT-3101 refactor and unblocking canonical/snapcraft#4908.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [x] Have you followed the guidelines for contributing?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] Have you successfully run `tox`?
-----
1. Refactor `RemoteBuild` errors to subclass `CraftError`
2. Slight changes to how remote build error are presented (moving information from `details` to `brief`)
3. Release notes for 4.2.4
Unblocks https://github.com/canonical/snapcraft/issues/4908
(CRAFT-3101)
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged craft-application#482 as be2750d8, making RemoteBuild errors CraftError subclasses, moving details into brief messages, and documenting release 4.2.4. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged craft-application#482 as be2750d8, making RemoteBuild errors CraftError subclasses, moving details into brief messages, and documenting release 4.2.4. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in canonical/craft-application#482 (be2750d8): remote-build errors now subclass CraftError, error details moved to brief, and 4.2.4 release notes were added. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged as be2750d8: RemoteBuild errors now subclass CraftError, presentation details move to brief, and 4.2.4 release notes were added, unblocking Snapcraft#4908. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged after reviewer approval and CI checks. Refactors RemoteBuild errors to subclass CraftError, improves error presentation by moving details to brief, and adds release notes for 4.2.4. Unblocks issue #4908. |
Update history
No update history recorded yet.
Related work
-
Related To:
canonical/snapcraft#4908
(confidence 100%)
The merged PR explicitly states that it unblocks this Snapcraft issue.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #5052 fix: catch remote build errors | snapcraft | merged | Merged in #5052 (commit 741e36eb9), updating craft-application to 4.2.4 so remote-build errors are caught as CraftErrors and include documentation links; fixes #4908. | |
| #4908 remote build: stop raising internal errors | snapcraft | closed | Remote-build errors now produce actionable messages and documentation links via merged PR #5052 (741e36eb9), updating craft-application to 4.2.4. | |
| #5223 remote-build: refactor onto craft-application's command | snapcraft | closed | Remote-build was refactored onto craft-application's RemoteBuild in merged PR #5239 (commit f31d99498), bringing the requested upstream improvements. | |
| #5239 refactor: use craft-application's remote-build | snapcraft | merged | Merged PR #5239 as f31d99498, replacing Snapcraft’s legacy remote-build path with craft-application’s command for CRAFT-4044. |