Improve the store errors returning exceptions
Metadata
Current evaluation
Merged changes improving store error handling by returning exceptions. Approved by maintainer come-maiz following multiple retest requests. Modifies 5 files with 93 additions and 61 deletions.
Suggested action: —
No scores available.
Issue body
LP: #1594636
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged changes improving store error handling by returning exceptions. Approved by maintainer come-maiz following multiple retest requests. Modifies 5 files with 93 additions and 61 deletions. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Improved store error handling by returning exceptions. Merged after multiple retest requests. Resolves LP #1594636. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged after multiple retest requests, improving store error handling by returning exceptions. Resolves LP #1594636. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2505 Test store | snapcraft | merged | Merged following maintainer approval. The pull request introduced 16 additions and 8 deletions across one file, passed CI checks, and was integrated into the main branch after a single review. | |
| #3466 store: do not unnecessarily catch/rewrite exceptions | snapcraft | merged | Merged into master. Removes unnecessary exception handling in store module login functions to surface original errors for debugging. Drops unused error constants and updates tests. Approved with passing CI and 100% diff coverage. | |
| #2749 storeapi: add StoreErrorList to handle store errors | snapcraft | merged | Added StoreErrorList class to centralize store error handling. Updated StorePushError to correctly parse resource-not-found and resource-forbidden responses, displaying clear user messages. Merged after reviewer approval, changing six files (+106/-42). | |
| #89 Store release | charmcraft | merged | Merged after approval by two reviewers. The pull request introduces Store release functionality, modifying 5 files with 196 additions and 1 deletion. No further discussions occurred. |