← Back to issue list

chore!: remove deprecated commands

View original Github issue

Metadata

Project
snapcraft
Number
#6035
Type
pull request
State
merged
Author
mr-cal
Labels
PR: Run Manual Spread
Created
Updated
Closed

Current evaluation

Merged after maintainer approval and a clean rebase. The changes remove deprecated commands and options, resolving issue #5674.

Suggested action:

No scores available.

Issue body

Removes deprecated commands and options. Fixes #5674 (SNAPCRAFT-1196) --- - [x] I've followed the [contribution guidelines](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md). - [x] I've signed the [CLA](http://www.ubuntu.com/legal/contributors/). - [x] I've successfully run `make lint && make test`. - [ ] I've added or updated any relevant documentation. - [ ] I've updated the relevant release notes.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged after maintainer approval and a clean rebase. The changes remove deprecated commands and options, resolving issue #5674.
qwen3.6-35b-a3b-mtp-q6 Merged: removed deprecated commands and options, resolving issue #5674. Author force-pushed for a clean rebase prior to merge. Linting and tests passed.
qwen3.6-35b-a3b-mtp-q6 Merged removal of deprecated commands and options, resolving #5674 and SNAPCRAFT-1196. Author force-pushed for a clean rebase before final merge following successful lint and test runs.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#5524 chore: add deprecation warnings for hidden commands snapcraft merged Merged after approval by two reviewers. The update adds uniform deprecation warnings for hidden commands and redirects their reference pages, resolving issue #5428 across 17 files.
79%
#1077 chore!: remove managed commands craft-application merged Merged a breaking change removing unused managed commands to finalize the migration to ProviderService. Resolves #1062. Approved by three reviewers with all CI checks passing prior to integration.
78%
#122 chore: eliminate pydantic warnings craft-archives merged Merged after two approvals and passing CI. Refactored three files to eliminate pydantic deprecation warnings, adding 24 lines and removing 15.
76%
#1538 chore: replace most lifecycle commands with upstream ones charmcraft merged Merged. Replaced most lifecycle commands with upstream versions, removing 297 lines across two files. Approved by three reviewers. Unit tests were not removed since only the pack command had substantial tests, which still requires upstream extension.
76%
#5540 chore: resolve deprecation warnings snapcraft merged Merged to resolve deprecation warnings before they become errors. Updates test mocking context manager setup. Approved by two reviewers and merged despite one failing integration test. Closes #5539.
76%
#5731 chore(legacy-cli): remove unused help.py and version.py, drop related test snapcraft closed Superseded by PR #5777, which combined this cleanup with #5773. The unused legacy help.py and version.py modules and their tests were removed, and the contributor received credit in the merged PR.
75%
#5994 chore!: remove legacy remote builder snapcraft merged Merged changes removing the legacy remote builder, resolving issue #5676. Approved by three reviewers and passing CI checks. The update deletes 3,200 lines across 30 files to clean up deprecated build infrastructure.
75%
#5777 chore(legacy-cli): remove unused help and version commands snapcraft closed Merged after confirming the legacy CLI entrypoint was never exposed in official releases. The unused help and version commands were safely removed, finalizing the cleanup of deprecated legacy code.
75%
#6010 chore!: remove remote-build service override snapcraft merged Merged cleanup removing the remote-build service override and manual spread tests. This follow-up addresses overlooked work from PR #5994. Approved by two reviewers, modifying seven files.
75%
#5094 chore: fix deprecation warnings snapcraft merged Merged after approval by two reviewers. The change resolves deprecation warnings from external libraries by updating code across five files, ensuring compatibility and preventing future runtime issues.
75%