← Back to issue list

feat: add CraftCommandError

View original Github issue

Metadata

Project
craft-cli
Number
#264
Type
pull request
State
merged
Author
tigarmo
Labels
Created
Updated
Closed

Current evaluation

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.

Suggested action:

No scores available.

Issue body

This new error class has a ``stderr`` attribute that holds the error output of an executed command. As the class' docstring indicates, this new attribute is halfway between ``brief`` and ``details``, verbosity-wise. It's meant to be used in cases where this error output is expected to be useful enough to the end user to make up for the extra text, even in "brief"-mode executions. Fixes #260 - [ ] Have you followed the guidelines for contributing? - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `tox`?

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b 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.
qwen3.6-35b-a3b-mtp-q6 Merged CraftCommandError class with a stderr attribute for command execution output. Provides intermediate verbosity between brief and details modes, resolving issue #260.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#363 feat: many improvements to error handling/reporting craft-application merged Merged after resolving a craft-cli dependency. Adds error handling improvements: standard exceptions, normalized validation messages, BuildPlanner updates, and doc URLs. Approved by two reviewers and passed CI.
72%
#624 Massive renaming of CommandError to CraftError (CRAFT-709). charmcraft merged Merged renaming CommandError to CraftError across 36 files. Imports updated to craft_cli and the old definition removed. A CharmcraftError subclass suggestion was declined, as CraftError already provides sufficient functionality.
72%
#3531 cli: warn and error messages to stderr (CRAFT-134) snapcraft merged 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.
70%