← Back to issue list

chore: add deprecation warnings for hidden commands

View original Github issue

Metadata

Project
snapcraft
Number
#5524
Type
pull request
State
merged
Author
mr-cal
Labels
PR: Squash
Created
Updated
Closed

Current evaluation

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.

Suggested action:

No scores available.

Issue body

- [x] Have you followed the [guidelines for contributing](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md)? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] Have you successfully run `make lint`? - [x] Have you successfully run `make test`? --- #### Changes * Adds a uniform deprecation warning for all deprecated commands. * Hides and redirects reference pages for hidden commands. #### What is not covered * A deprecation warning for the default command (`snapcraft` defaulting to `snapcraft pack`). See #5525. Fixes #5428 (SNAPCRAFT-1048)

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b 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.
qwen3.6-35b-a3b-mtp-q6 Merged as a squash commit. Adds uniform deprecation warnings for deprecated commands and hides or redirects reference pages for hidden commands, resolving issue #5428.
qwen3.6-35b-a3b-mtp-q6 Merged via squash to add uniform deprecation warnings for hidden commands and redirect their reference pages, resolving #5428. Default command deprecation is deferred.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#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.
81%
#6035 chore!: remove deprecated commands snapcraft merged Merged after maintainer approval and a clean rebase. The changes remove deprecated commands and options, resolving issue #5674.
79%
#5523 chore: add deprecation warning for legacy remote builder snapcraft merged Merged changes adding a deprecation warning for core22 snaps using the legacy remote builder. Approved by three reviewers with passing CI checks. Resolves issue #5427.
75%
#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.
75%
#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.
73%
#5281 chore: clean up some warnings snapcraft merged Merged a cleanup commit series resolving deprecation warnings across 21 files, cutting test output from over 3000 to 700. Risky changes and upstream warnings were excluded. Approved by two reviewers and passed CI.
73%
#794 chore: remove pydantic warnings craft-parts merged Successfully merged after approval by two reviewers and passing CI checks. The changes remove pydantic warnings across 9 files, adding 30 lines and deleting 16 to clean up framework deprecation notices.
73%
#650 chore: mark redundant functions as deprecated. craft-application open Marks redundant functions as deprecated across 15 files. Open 563 days with no reviews, comments, or CI. Deprecation work appears superseded by subsequent PRs (#660, #768, #1078) and codebase has evolved through multiple major versions.
72%
#2861 chore: remove prime key deprecation warning charmcraft merged Merged following two reviewer approvals. Removes the prime key deprecation warning, resolving issue #1786. The change deletes 24 lines in a single file and passed CI checks.
71%
#158 chore: update docstrings debcraft merged Merged updates to public function docstrings, adding missing parameters and return values. Approved by two reviewers, passed CI checks, and modified 10 files with 55 additions and 9 deletions.
70%