Recommend snap in the README.
Metadata
Current evaluation
Merged PR #396 (commit 90019231, merged as 1774741e) by maintainer facundobatista made a 3-line README edit recommending snap as the preferred install method for Charmcraft; approved by 2 reviewers.
Resolution: closed resolved
Reason: PR #396 was merged into master via commit 1774741e, applying the branch commit 90019231 "Recommend snap in the README." The diff (+3/-3 in README.md) added snap install guidance, approved by two reviewers with no unresolved comments.
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 #396 (commit 90019231, merged as 1774741e) by maintainer facundobatista made a 3-line README edit recommending snap as the preferred install method for Charmcraft; approved by 2 reviewers. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #396 (commit 1774741e), adding a README recommendation to use snap. The three-line documentation change was approved by two reviewers. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged a README update recommending snap. The change modified three lines across one file and was approved by two reviewers before integration. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged documentation update to recommend the snap package manager in the README. The maintainer integrated the change directly without review comments. |
Update history
No update history recorded yet.
Related work
-
Related To:
canonical/charmcraft#407
(confidence 30%)
Later README commit '378f2271 Update snap install to use --classic' refined this same installation section; current README still recommends snap install charmcraft --classic.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #116 include install info in README | charmcraft | merged | PR #116 (commit 9bd7d18f, "include install info in README") merged into master via merge commit 039b28dc, adding snap install/beta-channel instructions to Charmcraft's README. | |
| #41 package it as a snap | charmcraft | merged | Merged PR #41 added Charmcraft snap packaging in commit 0efe1c0a, with 69 lines across two files. | |
| #436 Improve the "we recommend using charmcraft from snap" story | charmcraft | closed | Closed after the CHARMCRAFT_DEVELOPER requirement was removed, eliminating the pip-install caveat that the proposed warning and documentation were meant to address. | |
| #327 Fix instructions to install from stable. | charmcraft | merged | Merged #327, updating Charmcraft's installation instructions to install the stable Snap release; merged in commit e8d67aa2. | |
| #303 snap: explicitly use requirements.txt for dependencies | charmcraft | merged | Merged PR #303 made the Charmcraft snap explicitly install dependencies from requirements.txt, via commit 17741dc7 and merge commit 83fbd200. | |
| #1950 fix: make charmcraft work with snap parallel installs | charmcraft | merged | Merged PR #1950 via commit 2399ddf8, fixing Charmcraft conflicts during parallel snap installs and adding spread regression coverage for issue #1770. |