← Back to issue list

chore!(models): remove BuildInfo and BuildPlanner

View original Github issue

Metadata

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

Current evaluation

Merged as commit 3c9170a4 (2 approvals, +36/-430): breaking change removing the BuildInfo and BuildPlanner models, documented in the 5.0.0 changelog. Requires #666.

Resolution: closed resolved

Reason: PR #667 was merged into main as commit 3c9170a4, deleting BuildPlanner/BuildInfo and noting in docs/reference/changelog.rst for 5.0.0 that BuildPlanner is replaced by BuildPlanService and internal BuildInfo by craft_platforms.BuildInfo.

Evaluated by: mr-cal-bot

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 `make lint && make test`? - [x] Have you added an entry to the changelog (`docs/reference/changelog.rst`)? --- Requires #666 Lint failures are pyright, test failures are a test that will be removed in a future PR.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged as commit 3c9170a4 (2 approvals, +36/-430): breaking change removing the BuildInfo and BuildPlanner models, documented in the 5.0.0 changelog. Requires #666.
stealth/space-bunny-alpha — closed resolved Merged PR #667 as commit 3c9170a4, removing BuildInfo and BuildPlanner after #666 introduced the replacement build-plan service.
qwen/qwen3.6-35b-a3b — — Merged a breaking change removing the BuildInfo and BuildPlanner models across nine files, deleting 430 lines and adding 36. Approved by two reviewers and passed CI checks.

Update history

No update history recorded yet.

Related work

  • Blocked By: canonical/craft-application#666 (confidence 97%)

    Explicit prerequisite ('Requires #666'): feat!(services): build plan service, merged as 621d1a32, introduced BuildPlanService that replaces BuildPlanner.

  • Likely Fixed By: canonical/craft-application#666 (confidence 90%)

    The replacement API (BuildPlanService plus craft_platforms.BuildInfo) landed here, making the legacy models removable.

Related issues

No related issues found above the similarity threshold.