← Back to issue list

godeps plugin: support for go-packages.

View original Github issue

Metadata

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

Current evaluation

Merged into master, adding go-packages support to the godeps plugin to match the go plugin. Approved by a reviewer, passed CI checks, and increased overall code coverage.

Suggested action:

No scores available.

Issue body

This is the on-par support for what is in the go plugin. LP: #1657212 Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged into master, adding go-packages support to the godeps plugin to match the go plugin. Approved by a reviewer, passed CI checks, and increased overall code coverage.
qwen3.6-35b-a3b-mtp-q6 Merged into master. Adds go-packages support to the godeps plugin, aligning it with the go plugin. Maintainer-authored and passed code coverage checks.
qwen3.6-35b-a3b-mtp-q6 Merged into master, adding go-packages support to the godeps plugin to align with the go plugin. The change passed code coverage checks and resolved Launchpad bug #1657212.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#2328 godeps plugin: support for bases snapcraft merged Merged. Updated the godeps plugin to prioritize the Go snap and added support for installing golang-go via build-packages on supported bases. Approved, passed CI, and integrated.
84%
#1179 godeps plugin: add git to build-packages snapcraft merged Merged a patch adding git to the godeps plugin build-packages. The change resolves a missing dependency for go get commands. Approved by one reviewer and merged directly without CI checks.
84%
#1046 godeps plugin: work when GOBIN is set snapcraft merged Merged into master. Resolves LP: #1656118 by fixing the godeps plugin to function correctly when GOBIN is set. Approved by one reviewer, passed CI, and modified two files (+15/-1 lines).
78%
#99 go plugin: implement a go-packages plugin property snapcraft merged Successfully merged implementation of a go-packages plugin property for the Go plugin. Approved with a minor comment, the change extends plugin configuration by adding 100 lines across 3 files.
73%
#691 Add godeps plugin. snapcraft merged Merged the godeps plugin for Snapcraft, resolving Launchpad bug #1595981. The change simplifies snapcraft.yaml for Go projects like Juju and adds tag support. Approved and verified by multiple contributors prior to integration.
72%
#2200 tests: add spread suite for godeps plugin snapcraft merged Merged changes move Godeps plugin integration tests to a Spread suite, resolving Launchpad bug #1783833. Approved by one reviewer, passed CI, and updated 12 files. Codecov noted a minor 0.08% coverage decrease.
71%
#2323 go plugin: support for bases snapcraft merged Merged. Introduces base support to the Go plugin, prioritizing the Go snap from the Snap Store and enabling golang-go installation on supported bases. Approved by reviewer and passed all CI checks.
70%
#1748202 go plugin doesn't honor godeps file snapcraft (launchpad) closed The go plugin fails to parse Godeps files for dependency versioning, causing build errors. The issue was closed without resolution, indicating it was abandoned.
70%