New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Switch to use Go modules #553
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
馃幐馃幎
Move as much stuff as possible from scripts to the main Makefile
Travis puts the source inside GOPATH, so GO111MODULE=auto ends with modules disabled.
I think it is ready for review now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice work. 馃 THx for taking care on all this tedious work and cleanup. 馃挊
If it works outside and inside $GOPATH we can merge it. 馃幐
Tested inside GOPATH with GO111MODULES=off |
Packages are also building correctly. They build with classical GOPATH + vendor/ by using |
Until now, we have been using glide to manage dependencies.
I don't think Go modules need an introduction, but here is a good place to start.
This PR moves the dependency handling to use Go 1.11 modules.
Worth mentioning, other providers like AWS already moved. It should not affect the end contributor, as we also vendor dependencies (
go mod vendor
), but it will give the ability to develop outside ofGOPATH
, which is very convenient.The PR also refactors the Makefile/travis chain in order to build using modules with the vendor directory inside travis. I also moved some of the lint/fmt checks from the travis scripts into the main Makefile after realizing the mess was so big having useless scripts inside travis/ that we were NOT actually running unit tests inside Travis 馃槧. We do now.
There are still a bunch of other scripts there to cleanup, so in a later PR I may end removing the travis directory completely.