(internal) Install snap via craft-providers API (CRAFT-1277)
Metadata
Current evaluation
Merged PR #891 as commit 456601ff, switching Charmcraft snap installation to the craft-providers API and completing CRAFT-1277.
Resolution: closed resolved
Reason: The approved PR was merged via commit 456601ff. Its implementation replaced Charmcraft's internal snap-install logic with the craft-providers API.
Evaluated by: mr-cal-bot
No scores available.
Issue body
No description provided.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #891 as commit 456601ff, switching Charmcraft snap installation to the craft-providers API and completing CRAFT-1277. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #891, replacing Charmcraft’s internal snap installation with the craft-providers API for CRAFT-1277; merge commit 456601ff. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #891, moving internal snap installation from shell commands to the craft-providers API for CRAFT-1277; merge commit 456601ff. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #891 switched Charmcraft's snap installation to the craft-providers API for CRAFT-1277 in merge commit 456601ff. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in canonical/charmcraft#891 via 456601ff, switching snap installation to the craft-providers API to resolve CRAFT-1277. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #891, switching snap installation to the craft-providers API and resolving CRAFT-1277; merge commit 456601ff. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged Charmcraft PR #891, replacing direct snap installation with the craft-providers API for CRAFT-1277; merge commit 456601ff. | |
| 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. |
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 PR #3933 (commit 929a26fd6) reworked snapcraft/providers to install snaps and packages via the craft-providers API instead of bespoke code (CRAFT-1281, CRAFT-1283). | |
| #71 providers: install rockcraft snap via craft-providers API | rockcraft | merged | Merged as f31183b7, refactoring provider setup to install the Rockcraft snap through the craft-providers API without functional changes. | |
| #522 providers: add support to install Charmcraft snap from store (CRAFT-502) | charmcraft | merged | Merged PR #522 added Snap Store installation for non-Linux targets and Linux opt-in via CHARMCRAFT_INSTALL_SNAP_CHANNEL, with channel selection and Windows test support. | |
| #402 providers: introduce charmcraft base configuration (CRAFT-261) | charmcraft | merged | Merged in charmcraft#402 via 481525e5, adding craft-providers and a Charmcraft buildd configuration that installs pip, setuptools, and injects the host Charmcraft snap. | |
| #1782778 Improve installation/injection logic for snapcraft (and core) | snapcraft (launchpad) | closed | Installation and host-injection logic was resolved in canonical/craft-providers PR #45, merged as commit 3573b67 under CRAFT-253. | |
| #622 Inject instanced host snaps to the provider | craft-providers | closed | Resolved downstream: craft-application added instanced-snap support in PR #467 (77a3582b), removing the need for a craft-providers API change. | |
| #143 build(deps): bump craft-providers to 1.20.0 | craft-application | merged | Merged PR #143 as commit f0349036, updating craft-providers to 1.20.0 so host-injected snaps also receive their base snap. | |
| #3943 providers: use new craft-providers interface | snapcraft | merged | Merged into snapcraft main as commit 2980d145c: removed in-tree Provider/LXDProvider/MultipassProvider classes in favour of craft-providers, and moved snapcraft/providers/providers.py to snapcraft/providers.py (+65/-1173). | |
| #2418 build(snap): enable internal plugins | charmcraft | merged | Merged in charmcraft#2418 via commit 4960244c, enabling internal Snapcraft plugins so relevant craft-application plugins can be used. |