Skip to content

Update dependencies, go.mod, go.sum #75

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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 10 commits into from
Sep 10, 2019
Merged

Conversation

butanian
Copy link
Contributor

@butanian butanian commented Sep 9, 2019

To update dependencies:
gorepoman -d vendor update go.opencensus.io
gorepoman -d vendor update aws-sdk-go
gorepoman -d vendor update github.com/golang/protobuf
gorepoman -d vendor update github.com/lib/pq
gorepoman -d vendor update google.golang.org/grpc
gorepoman -d vendor reconcile cloud.google.com/go

To update go.sum
go mod tidy

Edit: 9/10/2019
I also had to update one of the rows in the travis build matrix. When building with G 1.11 and modules, I had to specifically use a version higher than 11.3, to make building golang.org/x/sys/unix possible. Failure here: https://travis-ci.org/fullstorydev/hauser/jobs/583168386

Github issue that expands on the problem, and provides the alternative: golang/go#30446

@butanian butanian changed the title Update go.opencensus.io Update dependencies, go.mod, go.sum Sep 10, 2019
@butanian
Copy link
Contributor Author

@jameremo thank you for the help yesterday. PTAL

I also had to update the travis matrix to specifically use Go 1.11.4 when GO111MODULE is on. Updated the PR comment above with the reasons.

Copy link
Contributor

@jameremo jameremo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, nice to have the deps updated! 🎉

@butanian butanian merged commit b8be4d1 into fullstorydev:master Sep 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants