← Back to issue list

cli: warn and error messages to stderr (CRAFT-134)

View original Github issue

Metadata

Project
snapcraft
Number
#3531
Type
pull request
State
merged
Author
sergiusens
Labels
—
Created
Updated
Closed

Current evaluation

Merged after approval and passing CI checks. Redirects CLI warning and error messages to stderr, resolving CRAFT-134. Updates 7 files with 19 lines added and removed.

Suggested action: —

No scores available.

Issue body

Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com> - [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `./runtests.sh static`? - [ ] Have you successfully run `./runtests.sh tests/unit`? -----

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b — — Merged after approval and passing CI checks. Redirects CLI warning and error messages to stderr, resolving CRAFT-134. Updates 7 files with 19 lines added and removed.
qwen3.6-35b-a3b-mtp-q6 — — Merged changes that redirect CLI warning and error messages to stderr, resolving CRAFT-134. The update was integrated directly by the maintainer without additional comments.
qwen3.6-35b-a3b-mtp-q6 — — Merged. Redirects CLI warning and error messages to stderr, resolving CRAFT-134. Integrated by maintainer sergiusens without additional comments.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#591 Revisited all error usage to adapt to new craft-cli context. charmcraft merged Merged after approval and CI. Updated error handling for craft-cli by using from err for subprocess errors, adding details attributes, adjusting Store return codes, and transforming StoreServerError. Changed 8 files.
71%
#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.
70%
#635 Use all functionality from craft-cli (CRAFT-710). charmcraft merged Merged after approval. Integrated all craft-cli functionality, netting a reduction of 1724 lines across 11 files. Resolves CRAFT-710.
70%
#264 feat: add CraftCommandError craft-cli merged Merged implementation of CraftCommandError class with a stderr attribute for command output. Approved by two reviewers, passed CI checks, and resolves issue #260 across seven files.
70%