chore!: remove deprecated APIs
Metadata
Current evaluation
Merged, removing 8 deprecated APIs from craft-applications to resolve issue #1063. Approved by two reviewers with all CI checks passing. Locally verified with snapcraft, reducing code by 345 lines across 11 files.
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 `make lint && make test`?
- [x] Have you added an entry to the changelog (`docs/reference/changelog.rst`)?
---
Removes 8 deprecated APIs in craft-applications.
Apps that have already heeded the deprecation warnings should be unaffected.
I recommend reviewing per-commit.
Fixes #1063
(CRAFT-5151)
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged, removing 8 deprecated APIs from craft-applications to resolve issue #1063. Approved by two reviewers with all CI checks passing. Locally verified with snapcraft, reducing code by 345 lines across 11 files. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged removal of 8 deprecated APIs in craft-applications. Verified locally with snapcraft, requiring only a minor tweak to snapcraft try. Closes #1063. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1729 chore: update to craft-application 3 | charmcraft | merged | Merged after approval by two reviewers. Updated project dependencies to craft-application version 3 across 16 files, with 132 additions and 40 deletions. | |
| #463 chore(deps): update internal packages | rockcraft | merged | Merged automated dependency updates for craft-application, craft-cli, and craft-providers. Approved by two reviewers, passed CI checks, and integrated into main. Changes apply minor version bumps and internal fixes. | |
| #1765 chore: remove classic entrypoint | charmcraft | merged | Merged after approval by two reviewers. Removed a leftover classic entrypoint from the craft-application changeover, modifying seven files. | |
| #1327 chore(deps): update craft-application | charmcraft | merged | Merged dependency update for craft-application. Approved by two reviewers, modifying three files with minor adjustments (+11/-9 lines). Submitted by maintainer lengau. | |
| #4738 chore(deps): update dependency craft-application to v2.5.0 | snapcraft | merged | Merged automated dependency update upgrading craft-application from v2.4.0 to v2.5.0. Changes passed CI and received three approvals. The update includes fixes required for snapcraft 8.2.0. | |
| #6241 chore: remove snapcraft_legacy | snapcraft | merged | Merged removal of the snapcraft_legacy module, old specs folder, and dependencies after verifying no remaining entry points. Approved by two reviewers; noted unrelated integration test failures. | |
| #1617 chore(deps): update internal packages | charmcraft | merged | Merged automated dependency updates for craft-application to 2.1.0 and craft-parts to 1.29.0. Approved by two reviewers, passed CI, and merged with minimal changes. | |
| #1254 chore(deps): update internal packages | charmcraft | merged | Merged automated dependency updates for craft-cli, craft-parts, and snap-helpers. Approved by two reviewers and passed CI checks before integration. | |
| #2047 chore: remove deprecated craft-application calls | charmcraft | closed | Closed without merging. The pull request proposed removing deprecated craft-application calls by switching to service registration and replacing set_kwargs with update_kwargs across 25 files. Lacked reviewer engagement and CI checks, resulting in no action. | |
| #984 chore(deps): update internal packages | charmcraft | merged | Merged automated dependency updates for craft-parts, craft-providers, and snap-helpers. Approved by a reviewer and passed CI checks before integration. |