← Back to issue list

storeapi: rename SnapClientIndex to SnapAPI

View original Github issue

Metadata

Project
snapcraft
Number
#3448
Type
pull request
State
merged
Author
sergiusens
Labels
Created
Updated
Closed

Current evaluation

Merged changes renaming SnapClientIndex to SnapAPI to align with the api.snapcraft.io client interface. Docstrings were updated across six files. Approved by a reviewer and passed all CI checks.

Suggested action:

No scores available.

Issue body

This is the client side API on api.snapcraft.io. Rename and adapt docstrings to align with reality. 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 `./runtests.sh static`? - [ ] Have you successfully run `./runtests.sh tests/unit`? -----

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged changes renaming SnapClientIndex to SnapAPI to align with the api.snapcraft.io client interface. Docstrings were updated across six files. Approved by a reviewer and passed all CI checks.
qwen3.6-35b-a3b-mtp-q6 Merged renaming SnapClientIndex to SnapAPI within the storeapi module to accurately reflect the api.snapcraft.io client interface. Updated related docstrings to match the new naming convention.
qwen3.6-35b-a3b-mtp-q6 Merged. Renamed SnapClientIndex to SnapAPI in the storeapi module and updated docstrings to align with the api.snapcraft.io client-side API.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#1855 storeapi: add docstrings for _snap_index_client.py snapcraft merged Merged after approval by one reviewer and passing CI checks. Added docstrings to _snap_index_client.py, adding 32 lines. The external contributor's documentation update was successfully integrated.
82%
#3686 storeapi: export SnapAPI snapcraft merged Merged into snapcraft/7.0 after maintainer approval. Exports the SnapAPI from the storeapi module across four files. CI passed, review approved, and code coverage increased by 0.05%.
75%
#3450 storeapi: rename SCA to DashboardAPI snapcraft merged Merged. Renamed SCA to DashboardAPI in the storeapi module to align with the dashboard.snapcraft.io publisher API. Approved by one reviewer, passed CI checks, and modified nine files.
74%
#3592 storeapi: auth client for api.snapcraft.io snapcraft closed Merged into master. The pull request adds an authentication client for api.snapcraft.io with a single-line change. Codecov reported 100% diff coverage with no overall coverage impact. Closed after successful merge.
73%
#1793 project: refactor storeapi snapcraft merged Merged refactoring that split three snapcraft.storeapi classes from __init__.py into separate modules. Approved by a reviewer, passed CI, and resolved commit sync and dependency issues prior to merge.
71%