← Back to issue list

(internal) Install snap via craft-providers API (CRAFT-1277)

View original Github issue

Metadata

Project
charmcraft
Number
#891
Type
pull request
State
merged
Author
facundobatista
Labels
Created
Updated
Closed

Current evaluation

Merged changes to install snap via the craft-providers API, resolving CRAFT-1277. Updates six files with a net reduction of 145 lines. Approved by one reviewer with no unresolved comments or CI failures.

Suggested action:

No scores available.

Issue body

No description provided.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged changes to install snap via the craft-providers API, resolving CRAFT-1277. Updates six files with a net reduction of 145 lines. Approved by one reviewer with no unresolved comments or CI failures.
qwen3.6-35b-a3b-mtp-q6 Merged internal implementation to install snaps via the craft-providers API, resolving CRAFT-1277. The maintainer-submitted update was integrated directly without review comments or revisions.
qwen3.6-35b-a3b-mtp-q6 Merged implementation to install snaps via the craft-providers API, resolving internal ticket CRAFT-1277. Integrated directly without additional review comments or modifications.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#3933 providers: install snaps and packages via craft-providers API snapcraft merged Merged implementation for installing snaps and packages via the craft-providers API. Approved, passed CI, and increased unit test coverage by 0.46 percent across three changed files.
86%
#71 providers: install rockcraft snap via craft-providers API rockcraft merged Merged a refactor updating snapcraft to install the rockcraft snap via the new craft-providers API. The change modifies six files with no functional impact, passed CI checks, and received approval.
81%
#1098 build: update craft-providers rockcraft merged Merged update to the craft-providers dependency to resolve recurring snapd watch errors in spread tests. Approved by two reviewers and passed CI checks. The change modifies three files with minor adjustments.
76%
#143 build(deps): bump craft-providers to 1.20.0 craft-application merged Merged dependency update bumping craft-providers to 1.20.0. Approved by two reviewers, passed CI checks, and maintained 99.02% coverage. The change ensures host-injected snaps include their base snap in the instance.
74%
#4 snap: introduce initial snap packaging (CRAFT-578) rockcraft merged Merged initial snap packaging implementation (CRAFT-578). Approved by one reviewer, passed CI checks, and added 86 lines across three files.
74%
#5873 build(deps/auto): update craft-providers snapcraft merged Merged an automated dependency update for craft-providers to restore snapcraft compatibility. Applied via uv lock, approved by two reviewers, and integrated after passing CI checks.
73%
#3752 requirements: update dependencies snapcraft merged Merged into main following reviewer approval and CI checks. Updates dependencies, bumping craft-providers to 1.3.0 and craft-parts to 1.7.1 to resolve snap injection and plugin validation dependency issues.
73%
#164 dependencies: update craft-providers to 1.6.2 rockcraft merged Merged update to craft-providers 1.6.2, disabling automatic snap refreshes inside instances. Approved by three reviewers, passed CI, and merged with minimal changes across three files.
72%
#661 feat(provider): add snap registration api craft-application merged Merged addition of a snap registration API enabling providers to install extra snaps before lifecycle execution. Approved by two reviewers, changed three files with 110 additions and one deletion, resolving CRAFT-4174.
71%
#3715 plugs: install snaps from content plugs snapcraft merged Merged into snapcraft/7.0. Adds support for installing snaps via content plug default-provider fields. Approved by reviewer, passed all CI checks, and achieved 100% diff coverage across six modified files.
71%