← Back to issue list

feat: use craft-platforms build planning

View original Github issue

Metadata

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

Current evaluation

Closed unmerged after team discussion; the redesign was later delivered separately by #666 using craft-platforms, followed by #667 removing BuildPlanner.

Resolution: closed superseded

Reason: PR #501 was closed unmerged because the work would be handled separately. Later, #666 landed as commit 621d1a32 with a craft-platforms-backed BuildPlanService, and #667 (3c9170a4) removed the old BuildPlanner.

Evaluated by: mr-cal-bot

No scores available.

Issue body

This also fully deprecates BuildPlanner classes, but doesn't remove them. - [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`? -----

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed superseded Closed unmerged after team discussion; the redesign was later delivered separately by #666 using craft-platforms, followed by #667 removing BuildPlanner.
stealth/space-bunny-alpha — closed superseded Closed unmerged after team discussion; the migration was later handled by the build-plan service merged in #666 via commit 621d1a32.
stealth/space-bunny-alpha — closed stale Closed without merging after team discussion; the maintainer said craft-platforms build planning would be handled separately, with no replacement linked.
qwen/qwen3.6-35b-a3b — — Closed without merging. The author closed the pull request following a team discussion, noting that the build planning feature will be handled separately.

Update history

No update history recorded yet.

Related work

Related issues

Issue Project State Summary Similarity
#403 [PROTOTYPE] refactor: use craft-platforms craft-application closed Closed unmerged as a prototype; maintainer redirected the craft-platforms build-planner integration to #391, with the build-plan service later landing via #666 (621d1a32).
80%
#391 Use craft-platforms for build plans craft-application closed Build planning and filtering moved to craft-platforms in merged PR #25 (d9ff004); craft-application adopted the planner through merged PR #666 (621d1a32).
78%
#698 Use craft-platforms for build plans rockcraft closed Merged PR #715 (516d29e5) replaced Rockcraft's custom build-plan generation with craft-platforms, resolving #698 and aligning plans with downstream consumers.
75%
#5111 feat: use `craft-platforms` for build plans snapcraft merged Merged in #5111 via commit 7969ee8d5, replacing Snapcraft’s custom build-plan logic with craft-platforms utilities.
74%
#662 [PROTOTYPE] refactor: use craft-platforms rockcraft closed Prototype PR #662 was closed unmerged in August 2024; the production build-planner migration later landed via rockcraft#715 at commit 516d29e5.
74%
#639 Move the build planner logic into craft-platforms rockcraft closed Build planner logic extracted to craft-platforms in merged PR canonical/craft-platforms#25 (commit d9ff004); rockcraft later simplified in commit 5f898b32. Maintainer mr-cal confirmed completion.
73%
#1894 refactor: use craft-platforms for build plans charmcraft merged Merged PR #1894 (commit f1755e8f, +25/-33) replaced Charmcraft's custom build-plan generation for `platforms` charms with craft-platforms, routing exhaustive build plans through craft-application's BuildPlanService.
73%
#1876 Use craft-platforms for build plans charmcraft closed Closed as done: charmcraft PR #1894 (commit f1755e8f) switched build plan generation to craft-platforms via craft-application's BuildPlanService; rockcraft did the same in #715.
72%
#326 feat: add Platform model and default BuildPlanner craft-application merged Merged PR #326 as commit 4833c0e9, adding the Platform model and default BuildPlanner; the feature shipped in craft-application 3.0.0.
72%
#5015 Use craft-platforms for build plans snapcraft closed Snapcraft adopted craft-platforms build plans in merged PR #5111 (commit 7969ee8d5), replacing its custom planning logic.
72%