← Back to issue list

base client: support revisions

View original Github issue

Metadata

Project
craft-store
Number
#52
Type
pull request
State
merged
Author
sergiusens
Labels
—
Created
Updated
Closed

Current evaluation

Merged. Added revision support to the base client and updated integration tests to use a common fixture with integration servers. Approved, passed CI, and resolved CRAFT-949.

Suggested action: —

No scores available.

Issue body

Additionally, change integration tests to use common fixture and switch to the integration servers. Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com> - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? ----- CRAFT-949

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b — — Merged. Added revision support to the base client and updated integration tests to use a common fixture with integration servers. Approved, passed CI, and resolved CRAFT-949.
qwen3.6-35b-a3b-mtp-q6 — — Merged. Added revision support to the base client and updated integration tests to use a common fixture with integration servers. Resolves CRAFT-949.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#51 base client: support releasing craft-store merged Merged after two approvals, adding release support to the base client per CRAFT-950. The change modifies five files, introducing 189 lines and removing 2.
81%
#47 base client: improve doc strings (CRAFT-799) craft-store merged Improved doc strings for the base client. Merged after approval by one reviewer, passing CI checks, and modifying one file (+16/-12 lines). Signed off by maintainer sergiusens.
75%
#50 base client: support list-releases craft-store merged Merged after two reviewers approved, adding a list-releases namespaced endpoint with Pydantic validation to the base client. The update modified 13 files and resolved CRAFT-951.
73%
#56 Add a fuller integration test, and multiple minor changes to support it craft-archives merged Merged. Added a comprehensive integration test for repo.install() with minor supporting changes across seven files. Approved, passed CI, and achieved 100% diff coverage. Resolves CRAFT-1649 and CRAFT-1682.
71%