← Back to issue list

feat: add Platform model and default BuildPlanner

View original Github issue

Metadata

Project
craft-application
Number
#326
Type
pull request
State
merged
Author
mr-cal
Labels
Created
Updated
Closed

Current evaluation

Merged after two approvals. Adds a Platform model and default BuildPlanner. The maintainer coordinated timing with related remote-build PRs to bundle fixes in a craft-application hotfix.

Suggested action:

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`? ----- - Add a `Platform` model - Implement the default `BuildPlanner` (CRAFT-2824)

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged after two approvals. Adds a Platform model and default BuildPlanner. The maintainer coordinated timing with related remote-build PRs to bundle fixes in a craft-application hotfix.
qwen3.6-35b-a3b-mtp-q6 Merged after author delayed landing until dependent remote-build PRs were ready for a craft-application hotfix release. Adds a Platform model and default BuildPlanner.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#5111 feat: use `craft-platforms` for build plans snapcraft merged Merged the change to replace existing build plan logic with the craft-platforms library. Approved by two reviewers, the update modifies six files with a net reduction of 17 lines, streamlining platform detection and build configuration.
76%
#1894 refactor: use craft-platforms for build plans charmcraft merged Merged a refactor replacing custom build plan logic with craft-platforms for platforms charms. Approved by two reviewers, the change modifies two files with a net reduction of eight lines, streamlining platform handling.
74%
#362 feat: support platforms for the build plans rockcraft merged Merged feature adding platform support for build plans. Approved by two reviewers, the change modifies eight files (+39/-32 lines) and integrates the functionality.
73%
#639 Move the build planner logic into craft-platforms rockcraft open Refactor to move build planner logic into craft-platforms, requested by jugmac00. Work completed: craft-application#391 closed as resolved, rockcraft PR #715 and commit 5f898b32 implemented the change. No BuildPlanner class remains in codebase.
72%
#567 refactor: use default BuildPlanner rockcraft merged Merged to adopt the default BuildPlanner from craft-application. Approved by two reviewers after verifying upstream compatibility. Minor pydantic validator limitations were noted but did not block integration.
71%
#391 Use craft-platforms for build plans craft-application closed Build planner logic and filterer were upstreamed to craft-platforms via PR #25. Downstream integration is being prototyped in craft-application. The issue is closed as resolved.
71%