cli: enable craft-store logging
Metadata
Current evaluation
Enabled craft-store logging in the CLI. Merged after reviewer approval and passing CI. The change modifies a single line in one file, maintaining full code 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 `make lint`?
- [ ] Have you successfully run `pytest tests/unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Enabled craft-store logging in the CLI. Merged after reviewer approval and passing CI. The change modifies a single line in one file, maintaining full code coverage. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into main. Enables craft-store logging in the CLI. Maintainer sergiusens submitted the change, which achieved 100% diff coverage with no impact on overall project coverage. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into main. Enables craft-store logging in the CLI. Achieved 100% diff coverage with no change to overall project coverage. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #3692 legacy storeapi: use craft-store | snapcraft | merged | Merged PR replaces the legacy storeapi HTTP client with craft-store, removes obsolete tests and logout calls, and adapts the codebase to craft-store API changes. Approved and passed CI. | |
| #671 snap: update craft-store to 2.0.1 | charmcraft | merged | The craft-store update to 2.0.1 was successfully merged after receiving reviewer approval and passing CI checks. The minor two-file change was integrated without further modifications. | |
| #591 Revisited all error usage to adapt to new craft-cli context. | charmcraft | merged | Merged after approval and CI. Updated error handling for craft-cli by using from err for subprocess errors, adding details attributes, adjusting Store return codes, and transforming StoreServerError. Changed 8 files. | |
| #3862 store: move package out of commands | snapcraft | merged | Merged a pull request that moved the store package out of the commands directory. Approved by one reviewer, passed CI checks, modified 24 files, and resolves CRAFT-1240. | |
| #574 requirements: update craft-store and others | charmcraft | merged | Merged dependency updates for craft-store and other packages. Approved by one reviewer with zero unresolved comments. Modified four files, adding and removing 23 lines each. |