feat: add a BuildPlan service
Metadata
Current evaluation
Merged into main as 01185c3bf, adding a BuildPlan service and create_launchpad_build_plan() for remote builds, with revised unit tests.
Resolution: closed resolved
Reason: PR #5368 was approved and merged into main in commit 01185c3bf.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [x] Have you followed the [guidelines for contributing](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md)?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] Have you successfully run `make lint`?
- [x] Have you successfully run `make test`?
---
* Adds a `BuildPlan` service
* Adds a new `create_launchpad_build_plan()` method
* This will be used by the remote-build command in CRAFT-4415
* Reworks the unit tests to exercise the new method and drops all `xfail`s
Creates https://github.com/canonical/craft-application/issues/719
(CRAFT-4417)
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged into main as 01185c3bf, adding a BuildPlan service and create_launchpad_build_plan() for remote builds, with revised unit tests. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #5368 as 01185c3bf, adding a BuildPlan service and create_launchpad_build_plan() for remote builds, with reworked unit tests and all xfails removed. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #5368 (commit 01185c3bf), adding a BuildPlan service and create_launchpad_build_plan() for remote builds; unit tests were reworked and all xfails removed. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged addition of a BuildPlan service and create_launchpad_build_plan() method to support remote builds. The change adds 847 lines and removes 125 across five files, updates unit tests, and received two reviewer approvals. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Introduces a BuildPlan service and create_launchpad_build_plan() method to enable remote builds. Unit tests were reworked and xfails removed. |
Update history
No update history recorded yet.
Related work
-
Related To:
canonical/craft-application#719
(confidence 95%)
Created by PR #5368 to coordinate the craft-application BuildPlan integration.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #666 feat!(services): build plan service | craft-application | merged | Merged BuildPlanService and related application, service, and testcraft changes in PR #666 (commit 621d1a32), resolving CRAFT-4159. | |
| #362 feat: support platforms for the build plans | rockcraft | merged | Merged PR #362, adding platform support to build plans; the change is recorded in commit 9cc5af95. |