← Back to issue list

feat!(services): build plan service

View original Github issue

Metadata

Project
craft-application
Number
#666
Type
pull request
State
merged
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

Merged BuildPlanService and related application, service, and testcraft changes in PR #666 (commit 621d1a32), resolving CRAFT-4159.

Resolution: closed resolved

Reason: PR #666 was merged into the main branch as commit 621d1a32, introducing the requested build plan service and related integration changes.

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [ ] Have you followed the guidelines for contributing? - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `make lint && make test`? - [ ] Have you added an entry to the changelog (`docs/reference/changelog.rst`)? --- Recommended review strategy (mark each file as viewed after completing): 1. Review [buildplan.py](https://github.com/canonical/craft-application/pull/666/files#diff-b18e5e1a75b585219477236f6de0036053b3482ccad4ab18878fadc4f6202548) and its related tests. 2. Review the changes to the other services. 3. Review the changes to the Application. 4. Review testcraft. 5. Review the remainder CRAFT-4159

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged BuildPlanService and related application, service, and testcraft changes in PR #666 (commit 621d1a32), resolving CRAFT-4159.
stealth/space-bunny-alpha — closed resolved Merged BuildPlanService and related application, service, and testcraft changes in PR #666 (commit 621d1a32), resolving CRAFT-4159.
qwen/qwen3.6-35b-a3b — — Merged the build plan service implementation, resolving CRAFT-4159. Approved by two reviewers and passing CI, the change spans 42 files with 1851 additions and 786 deletions.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#646 feat!(services): add a project service craft-application merged Merged PR #646 added ProjectService, refactored application/service wiring, and added documentation and tests; it landed as commit 16a20e36.
73%
#5368 feat: add a BuildPlan service snapcraft merged Merged into main as 01185c3bf, adding a BuildPlan service and create_launchpad_build_plan() for remote builds, with revised unit tests.
72%