-
Notifications
You must be signed in to change notification settings - Fork 21
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
Update Go versions and vendor using Godep (and fix the build) #331
Conversation
Move to the standard library context package
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.
LGTM, wonder if we could vendor the peg library too?
@@ -15,6 +15,7 @@ | |||
package command | |||
|
|||
import ( | |||
netcontext "context" |
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.
I think we could probably remove the netcontext
here?
@drcapulet I guess we could import the peg library somewhere (maybe in |
Godep will only vendor imported packages, not commands. I wouldn't want to roll something on our own here just to get a stable copy of That Travis CI build issue you mentioned is from 2015. How has this project not run into this problem before? |
Looking at previous builds, it looks like Cassandra has always failed to start up, but the tests still passed somehow. Maybe something in the newer versions of Go causes the tests to fail. |
All of this to get the build to pass...