← Back to issue list

docker: instructions to build from the snap

View original Github issue

Metadata

Project
snapcraft
Number
#1840
Type
pull request
State
merged
Author
sergiusens
Labels
Created
Updated
Closed

Current evaluation

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.

Suggested action:

No scores available.

Issue body

Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com> - [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] 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)? - [x] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)? - [x] Have you successfully run `./runtests.sh static`? - [x] Have you successfully run `./runtests.sh unit`? -----

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b 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.
qwen3.6-35b-a3b-mtp-q6 Documentation update adding Docker build instructions for the snap was merged. Maintainer sergiusens submitted the changes, passed static and unit tests, and the PR was accepted without additional review.
qwen3.6-35b-a3b-mtp-q6 Merged documentation update adding Docker build instructions for creating snaps. The maintainer integrated the guide after passing static and unit tests.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#3987 ci: remove docker before building snap snapcraft merged 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.
75%
#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.
74%
#2245 docker: support for testing snapcraft in proposed snapcraft merged Merged additions for a xenial-proposed snapcraft test Dockerfile and cleanup of the standard Dockerfile. Changes received approval, passed CI checks, and were validated through successful container execution.
74%