snap: pull early
Metadata
Current evaluation
Merged a one-line change to execute snapcraftctl pull before snapcraftctl set-version. Approved by one reviewer, passed CI checks, and modified a single file.
Suggested action: —
No scores available.
Issue body
Do "snapcraftctl pull" before "snapcraftctl set-version"
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/)?
- [ ] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)?
- [ ] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)?
- [ ] 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 a one-line change to execute snapcraftctl pull before snapcraftctl set-version. Approved by one reviewer, passed CI checks, and modified a single file. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a change to execute snapcraftctl pull before snapcraftctl set-version, ensuring correct build initialization order. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a change reordering snapcraft build steps to execute snapcraftctl pull before snapcraftctl set-version, correcting the command execution sequence. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #86 meta: use snapcraft's versioning scheme | starbase | merged | Merged after two approvals and passing CI. Updates the versioning scheme to match snapcraft's using setuptools-scm, adding 78 lines across two files with zero coverage impact. | |
| #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. | |
| #58 changes to snapcraft.yaml, and version, for 0.2.0 | charmcraft | merged | Merged following single reviewer approval. Updated snapcraft.yaml and bumped project version to 0.2.0 across two files, adding 107 lines and removing 9. | |
| #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. | |
| #228 packages: run snap operations silently | craft-parts | merged | Merged to fix #218 by running snap subprocesses silently, preventing stdout leakage that conflicts with craft-cli during lifecycle execution. Approved by one reviewer, passed CI, and modified two files (+32/-15 lines). | |
| #894 store: send snapcraft version in a header | snapcraft | merged | Merged. The snapcraft version is now sent in a request header, resolving Launchpad bug #1640501. Approved by one reviewer with zero unresolved comments. Changes modify four files, adding 21 lines and removing 23. | |
| #3208 snap: set PATH for snapcraft command | snapcraft | merged | Merged to configure the PATH for the snapcraft command in the build environment, ensuring parity with launchpad buildd and destructive-mode instances. Approved, passed CI, and modified three files. | |
| #2063 many: add snapcraftctl set-version | snapcraft | merged | Merged. Adds snapcraftctl set-version to configure snap versions from scriptlets, resolving issue 1676. Provides reusable infrastructure for future setter commands. Approved by reviewer and passed CI checks. | |
| #1407 recording: record the original snapcraft.yaml | snapcraft | merged | Merged to record the original snapcraft.yaml. Approved by one reviewer, modified 6 files with 91 additions and 31 deletions. Author noted the change requires PR #1406. | |
| #204 Update snapcraft.yaml | charmcraft | merged | Merged updates to snapcraft.yaml by markshuttle. Applied text changes for snapcraft.io and snap users, approved by two reviewers with zero unresolved comments. Diff added 18 lines and removed 14 across one file. |