You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository was archived by the owner on Sep 9, 2020. It is now read-only.
gps happens to require the 2.x branch of github.com/Masterminds/semver, which is not the master branch, but gps does not explicitly vendor its dependencies. That means go get github.com/sdboyer/gps (and its friends, like go test) don't work out of the box.
This took me about 30 minutes to suss out, which is a pretty significant hurdle to jumping in to help on gps/dep.
I'd be happy with loud documentation in CONTRIBUTING.md that says something like "You'll need to run glide install to populate /vendor/ before you can get started." I'd be even happier if the vendor directory were checked in, which would match what github.com/golang/dep does.
Yeah, this is definitely not the best. Let's keep it simple for now, with a note in CONTRIBUTING.md. gps is going to move directly into dep soon (#300), at which point we're probably going to be developing over there, so the considerations related to vendor will be different.
From @spenczar on March 10, 2017 14:49
gps
happens to require the 2.x branch of github.com/Masterminds/semver, which is not the master branch, but gps does not explicitly vendor its dependencies. That meansgo get github.com/sdboyer/gps
(and its friends, likego test
) don't work out of the box.This took me about 30 minutes to suss out, which is a pretty significant hurdle to jumping in to help on gps/dep.
I'd be happy with loud documentation in
CONTRIBUTING.md
that says something like "You'll need to runglide install
to populate/vendor/
before you can get started." I'd be even happier if the vendor directory were checked in, which would match what github.com/golang/dep does.Copied from original issue: sdboyer/gps#182
The text was updated successfully, but these errors were encountered: