Add Go modules to vendor directory#17
Add Go modules to vendor directory#17chris-crone wants to merge 3 commits intocompose-spec:masterfrom
Conversation
d684d51 to
c06ed03
Compare
But you have to checkout |
c06ed03 to
51a8c95
Compare
|
It actually appears that you do need the |
Signed-off-by: Christopher Crone <christopher.crone@docker.com>
Signed-off-by: Christopher Crone <christopher.crone@docker.com>
51a8c95 to
2601027
Compare
Signed-off-by: Christopher Crone <christopher.crone@docker.com>
thaJeztah
left a comment
There was a problem hiding this comment.
SGTM
@tonistiigi ptal (you dealt with go mod a bit more than I did)
|
|
||
| set -eu -o pipefail | ||
|
|
||
| go mod tidy 2>/dev/null && go mod vendor 2>/dev/null |
There was a problem hiding this comment.
One thing to consider (no blocker) is if we want this to always be run, or only if either the Go version is updated, or if go.mod or vendor/ has been modified in a pull request
There was a problem hiding this comment.
Tonis didn't have access to this GitHub org. I've added him and will re-ping when he's accepted.
Chatting to @thaJeztah and @justincormack, for things that we expect to ship as a binary it makes more sense to include the modules as vendored code.
This also speeds up the build for the CI as we don't need to fetch anything.