fix: add snap commands logging
Metadata
Current evaluation
Merged PR #401 added Snap command logging to expose command output and errors for debugging, fixing #396; landed as commit d7aa4dc.
Resolution: closed resolved
Reason: PR #401 was approved and merged into main; repository history records the change as d7aa4dc ("fix: capture details for snap errors"). Its description explicitly states that it fixes #396.
Evaluated by: mr-cal-bot
No scores available.
Issue body
For helps understanding error or debug.
Fix: https://github.com/canonical/craft-providers/issues/396
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #401 added Snap command logging to expose command output and errors for debugging, fixing #396; landed as commit d7aa4dc. | |
| qwen/qwen3.6-35b-a3b | — | closed resolved | Added snap commands logging to aid debugging, resolving #396. Merged as #401 (commit 8bb8647). |
Update history
| Date | Change |
|---|---|
| created |
Related work
-
Related To:
canonical/craft-providers#396
(confidence 100%)
The Snap command-output issue was explicitly fixed by merged PR #401.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #3683 commands: add snap command | snapcraft | merged | Merged #3683 restored the legacy snap packaging command and fixed output-parameter forwarding to managed instances (merge d5b3178ef). | |
| #13 fix: deref snap name and s/channel/release/ | starflow | merged | Merged PR #13 fixed dereferencing of the snap name and channel/release paths, landing as commit 7153678. | |
| #591 Improve error reporting | snapcraft | merged | Merged PR #591 as commit b9258ea73, making errors log before exit status 1 and adding a clear error for uploads of nonexistent snaps. | |
| #1325 cli: proper error for failed snap command | snapcraft | merged | Merged PR #1325 in commit 4f21e5153, making the CLI return a proper error code when a snap command fails. |