← Back to issue list

cli: include consistent commands to fix error conditions

View original Github issue

Metadata

Project
snapcraft
Number
#1790
Type
pull request
State
merged
Author
come-maiz
Labels
Created
Updated
Closed

Current evaluation

Merged implementation of consistent CLI commands to resolve error conditions. Closes #1733, depends on #1640, and received reviewer approval with minimal code changes across three files.

Suggested action:

No scores available.

Issue body

- [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)? - [x] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)? - [x] Have you successfully run `./runtests.sh static`? - [x] Have you successfully run `./runtests.sh unit`? ----- Closes #1733 Requires #1640

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged implementation of consistent CLI commands to resolve error conditions. Closes #1733, depends on #1640, and received reviewer approval with minimal code changes across three files.
qwen3.6-35b-a3b-mtp-q6 Merged to implement consistent CLI commands for resolving error conditions. Closes #1733 and depends on #1640. Code was reviewed and integrated into the main branch.
qwen3.6-35b-a3b-mtp-q6 Merged changes implementing consistent CLI commands to handle error conditions. Closes #1733 and depends on #1640. Approved and integrated into the codebase.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#1640 cli: add what, why, and how to fix to the common errors snapcraft merged Merged pull request adding what, why, and how to fix guidance to common CLI error messages. Approved by two reviewers, modified 15 files (+484/-155 lines), and passed CI after a temporary test failure was rerun.
77%
#1436 cli: properly handle exceptions snapcraft merged Merged a global exception handler for CLI errors, superseding PR #1429. Replaced per-command decorators to simplify maintenance and auto-coverage. Fixed exit_code property test failures before merge.
74%
#3125 cli: cleanup error handling snapcraft merged Merged CLI error handling cleanup. Removed legacy try/except block for Ubuntu 16.04 and switched to inline type hints to eliminate noqa comments. Approved by reviewer and passed CI checks.
70%