snap: do not refresh already installed snaps
Metadata
Current evaluation
Merged changes to prevent refreshing already installed snaps by removing two lines of code. Approved by one reviewer and integrated into the main branch.
Suggested action: —
No scores available.
Issue body
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 to prevent refreshing already installed snaps by removing two lines of code. Approved by one reviewer and integrated into the main branch. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into main branch. Implements logic to skip refreshing snaps that are already installed. Integrated by maintainer sergiusens without additional comments or revisions. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged pull request preventing snapcraft from refreshing already installed snaps. Authored by maintainer sergiusens, the change was accepted and integrated into the main branch without additional discussion. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #3618 snap: do not refresh already installed snaps | snapcraft | merged | Merged to prevent refreshing already installed snaps. Ported from #3617 to snapcraft_legacy, the approved change removes two lines of code. | |
| #161 snap: do not refresh already installed snaps | craft-parts | merged | Merged a port of snapcraft#3617 to craft-parts. The change prevents refreshing already installed snaps by removing two lines of code. Approved by one reviewer and passed CI checks. | |
| #3502 snaps: do not validate snaps before install/refresh (Fixes LP#1901733) | snapcraft | merged | Merged into master. Removes redundant snap validation before install and refresh, as the store API already handles branch checks. Approved by two reviewers, passed CI, and resolves Launchpad bug #1901733. | |
| #1692 fix: refresh build snaps if already installed | craft-parts | open | Refreshes build snaps when already installed to reflect channel changes, fixing #496. Has 2 approvals, 1 unresolved review comment, and mostly passing CI. | |
| #3102 build providers: prevent snap refreshing in build environment | snapcraft | merged | Merged to prevent snap refreshing in the build environment by holding refresh for one day. Adjusted per reviewer feedback to avoid incomplete manifests and address versioning string comparisons. Approved and merged. | |
| #5689 fix: remove unnecessary snapd_snap flag (#5316) | snapcraft | merged | PR #5316 merged, removing the unnecessary snapd_snap flag across two files and deleting 14 lines. The change received three approvals, passed all CI checks, and was successfully integrated. | |
| #3505 snaps: don't validate snaps before `SnapPackage.download()` | snapcraft | merged | Merged a fix that skips snap validation before SnapPackage.download(), resolving branch staging issues (LP#1901733). Approved by two reviewers, passed CI, and changed three files (+16/-11 lines). |