← Back to issue list

cli: catch craft-provider errors

View original Github issue

Metadata

Project
snapcraft
Number
#4295
Type
pull request
State
merged
Author
mr-cal
Labels
—
Created
Updated
Closed

Current evaluation

Merged #4295 as commit 511061252, adding CLI handling for craft-provider errors; the PR states it resolves #4262 (CRAFT-1898).

Resolution: closed resolved

Reason: PR #4295 was approved and merged into main as commit 511061252. Its two-file change catches craft-provider errors in the CLI, and its description explicitly links the resolution to #4262.

Evaluated by: mr-cal-bot

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] Have you successfully run `make lint`? - [x] Have you successfully run `pytest tests/unit`? ----- Resolves #4262 (CRAFT-1898)

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged #4295 as commit 511061252, adding CLI handling for craft-provider errors; the PR states it resolves #4262 (CRAFT-1898).
stealth/space-bunny-alpha — closed resolved Merged #4295 as commit 511061252, adding CLI handling for craft-provider errors; the PR states it resolves #4262 (CRAFT-1898).
stealth/space-bunny-alpha — closed resolved Merged #4295 as commit 511061252, adding CLI handling for craft-provider errors; the PR states it resolves #4262 (CRAFT-1898).
stealth/space-bunny-alpha — closed resolved PR #4295 merged into main, adding CLI handling for craft-provider errors and resolving #4262 (CRAFT-1898).
qwen/qwen3.6-35b-a3b — — Merged after approval, adding CLI error handling for craft-provider failures. Resolves #4262 and CRAFT-1898. Passed CI checks and achieved 100% diff coverage across two modified files.
qwen3.6-35b-a3b-mtp-q6 — — Merged into main. Implements CLI error handling for craft-provider failures, resolving #4262. Changes passed linting and unit tests with 100% coverage.

Update history

No update history recorded yet.

Related work

  • Related To: snapcore/snapcraft#4262 (confidence 100%)

    The merged PR explicitly states that it resolves this issue.

Related issues

Issue Project State Summary Similarity
#4419 cli: catch craft-provider errors snapcraft merged Merged into hotfix/7.5 as commit 6b2f6d207, adding CLI handling for craft-provider errors; it backported the #4295 fix for CRAFT-2174.
92%
#4262 LXD errors from Craft Providers not properly presented snapcraft closed Fixed by #4295 (commit 511061252), which catches Craft Providers errors and shows user-friendly messages instead of tracebacks; backported for 7.5 by #4419 (commit 6b2f6d207).
70%