cli: only say sorry for in-snapcraft issues
Metadata
Current evaluation
Merged into master after approval and passing CI. The change updates CLI error messaging to only apologize for in-snapcraft issues, removing leftover refactor code. The 3 additions and 2 deletions achieved 100% diff coverage.
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 into master after approval and passing CI. The change updates CLI error messaging to only apologize for in-snapcraft issues, removing leftover refactor code. The 3 additions and 2 deletions achieved 100% diff coverage. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into master. Updates CLI error messaging to only apologize for in-snapcraft issues, addressing a refactor leftover. Maintainer confirmed alignment with existing behavior. Merged with 100% diff coverage. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged CLI changes restricting error apologies to in-snapcraft issues. Maintainer confirmed it fixes a refactor leftover while preserving behavior. Passed tests with negligible coverage impact. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2469 cli: clean up snapcraft push output | snapcraft | merged | Merged cleanup of the snapcraft push CLI output. Verbose messages were moved to debug level, filename formatting was standardized, and log coloring was adjusted. Approved by one reviewer, passed CI, and integrated into the main branch. |