ci: remove docker before building snap
Metadata
Current evaluation
Merged into main after reviewer approval and passing CI checks. The change modifies four files to remove Docker before building the snap, with no impact on overall code coverage.
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 `make lint`?
- [ ] Have you successfully run `pytest tests/unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged into main after reviewer approval and passing CI checks. The change modifies four files to remove Docker before building the snap, with no impact on overall code coverage. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into main. Updates the CI pipeline to remove Docker prior to snap building, preventing environment conflicts. Code coverage remains stable at 91.04%. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into main. Updated CI workflow to remove Docker before building the snap package. Code coverage remained unchanged. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #3990 ci: remove docker before building snap | snapcraft | merged | Merged cherry-pick of commit 28f59d3 onto the hotfix/7.2 branch. The change removes Docker before building the snap in CI. Approved by one reviewer and passed all checks. | |
| #1840 docker: instructions to build from the snap | snapcraft | merged | Merged after approval and passing CI checks. Adds Docker build instructions for creating snaps across three documentation files, introducing 67 lines of guidance for users. | |
| #3210 docker: install snapd dependency | snapcraft | merged | Merged. Adds snapd installation to Docker images to satisfy snapcraft's new snap pack requirement. Approved by two reviewers, passed CI, and modified five files. | |
| #3449 ci: don't publish snap on push to master | snapcraft | merged | Merged a CI update removing automatic snap publishing on master pushes. The change deletes four lines of CI code as publishing is handled externally for all architectures. Approved and merged cleanly. | |
| #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. | |
| #1994 docker: add the architecture | snapcraft | merged | Configured SNAP_ARCH for Docker images and added explicit failure if missing. Merged following successful CI checks and two reviewer approvals. | |
| #2945 ci: only run docker builds for cron | snapcraft | closed | Merged a CI update that switches docker image builds from every pull request to a scheduled cron job. This reduces unnecessary CI overhead since docker builds rarely align with snapcraft source changes. | |
| #1803 ci: correctly run from snap | snapcraft | merged | Merged a CI fix to correctly run tests from snap by separating development and runtime dependencies. Approved by one reviewer, passed CI, and resolved dependency isolation to expose hidden bugs for future fixes. |