feat: use `craft-platforms` for build plans
Metadata
Current evaluation
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.
Suggested action: —
No scores available.
Issue body
- [ ] Have you followed the [guidelines for contributing](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md)?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `tox run -m lint`?
- [ ] Have you successfully run `tox run -e test-py310`? (supported versions: `py39`, `py310`, `py311`, `py312`)
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged to integrate the craft-platforms library for build plans. The change replaces legacy platform detection logic, standardizing build configurations. Approved and closed by maintainer mattculler. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged implementation of craft-platforms for build plans. Maintainer mattculler integrated the feature into the main branch, replacing legacy build plan logic with the new dependency. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #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. | |
| #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. | |
| #698 Use craft-platforms for build plans | rockcraft | closed | Closed and superseded by prototype PR #662, which implements craft-platforms for build plans to deduplicate logic and ensure consistency across projects. No explicit resolution or merge details are documented. | |
| #715 refactor: use craft-platform for the build plans | rockcraft | merged | Merged refactor to use craft-platform for build plans, creating a single source for Rockcraft and remote builds. Approved by two reviewers, resolving issue #698 with 31 additions and 3 deletions across five files. | |
| #1876 Use craft-platforms for build plans | charmcraft | closed | Closed without resolution. The proposal to use craft-platforms for build plans to deduplicate logic and ensure consistency was not implemented or superseded. | |
| #326 feat: add Platform model and default BuildPlanner | craft-application | merged | 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. | |
| #495 feat: add craft-platforms compatibility methods | craft-application | merged | Merged after approval by two reviewers and passing CI checks. Added craft-platforms compatibility methods and deprecation warnings. Documentation breakage was later resolved in PR #498. | |
| #501 feat: use craft-platforms build planning | craft-application | closed | Closed without merging. The author closed the pull request following a team discussion, noting that the build planning feature will be handled separately. | |
| #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. | |
| #5205 refactor: use craft-platforms utilities | snapcraft | merged | Merged following approval by two reviewers. Refactored the codebase to use craft-platforms utilities, resolving CRAFT-3278 and closing #4993. Net reduction of 307 lines across 20 files. |