cli: enable craft-store logging
Metadata
Current evaluation
Merged PR #3744 (commit 6f7e26e6a) enabled craft-store logging in the Snapcraft CLI; merge commit b5d3e916d.
Resolution: closed resolved
Reason: PR #3744 was approved and merged with commit 6f7e26e6a, recorded by merge commit b5d3e916d. The change enables craft-store logging in the CLI.
Evaluated by: mr-cal-bot
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 |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #3744 (commit 6f7e26e6a) enabled craft-store logging in the Snapcraft CLI; merge commit b5d3e916d. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #3744 (6f7e26e6a) enabled craft-store logging in the CLI; it was approved with 100% diff coverage. | |
| 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. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #3692 legacy storeapi: use craft-store | snapcraft | merged | PR #3692 merged into snapcraft/7.0 (merge 5a850ca04, commit 15b55daa8), swapping legacy storeapi's http_clients implementation for craft-store, dropping obsolete tests, adapting callers to API changes, and removing needless logout calls. | |
| #3794 store: full support for legacy login | snapcraft | merged | Merged PR #3794 via commit 86193a7b4 (7194ef18d), restoring legacy-login compatibility by using existing Snapcraft credentials to build craft-store authorization headers. | |
| #5084 build(deps): bump craft-store to 3.0.2 | snapcraft | merged | Merged as 1b5789fa8: upgraded craft-store to 3.0.2 and cherry-picked #5061, fixing #5077's core20 build failures; released in Snapcraft 8.4.2. | |
| #5536 fix(store): catch credential decoding errors | snapcraft | merged | Merged in canonical/snapcraft#5536 (commit 3784560f3), converting malformed store credential decoding failures into a user-friendly error and updating craft-store. | |
| #4863 fix(cli): properly capture store errors | snapcraft | merged | Merged #4863 in commit 426df7ec3, restoring legacy-style store-error capture so store failures are no longer reported as Snapcraft internal errors. | |
| #671 snap: update craft-store to 2.0.1 | charmcraft | merged | Merged into main as 26b067a7 (merge 23612dab), bumping craft-store from 2.0.0 to 2.0.1 in the snapcraft.yaml and requirements files (+2/-2 lines). |