← Back to issue list

errors: catch all CraftStoreErrors (CRAFT-813)

View original Github issue

Metadata

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

Current evaluation

Merged PR #670 broadened craft_store initialization error handling to catch all CraftStoreErrors; implementation fe3215e3 landed via merge commit 677b399f.

Resolution: closed resolved

Reason: PR #670 was approved and merged. Implementation commit fe3215e3 was incorporated by merge commit 677b399f.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Catch all CraftStoreErrors while initializing the craft_store library.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #670 broadened craft_store initialization error handling to catch all CraftStoreErrors; implementation fe3215e3 landed via merge commit 677b399f.
stealth/space-bunny-alpha — closed resolved Merged PR #670, broadening craft_store initialization error handling to catch all CraftStoreErrors. Merge commit: 677b399f; implementation commit: fe3215e3.
qwen/qwen3.6-35b-a3b — — Merged. Catches all CraftStoreErrors during craft_store library initialization. Approved by one reviewer with passing CI checks across 5 files (+35/-30).
qwen3.6-35b-a3b-mtp-q6 — — Merged changes to catch all CraftStoreErrors during craft_store library initialization, resolving CRAFT-813.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#61 errors: catch the correct JSONDecodeError craft-store merged Merged in craft-store#61 via commit e2ce4b5, correcting JSONDecodeError handling for CRAFT-1238 across three files.
76%
#12 errors: expose the StoreServerError cause from the returned json (CRAFT-582) craft-store merged Merged canonical/craft-store#12, exposing StoreServerError causes from returned JSON and adding expanded error handling and tests.
73%