square / metrics

Metrics Query Engine
Apache License 2.0
170 stars 21 forks source link

Update Go versions and vendor using Godep (and fix the build) #331

Closed DanielNi closed 7 years ago

DanielNi commented 7 years ago

All of this to get the build to pass...

DanielNi commented 7 years ago

@drcapulet I guess we could import the peg library somewhere (maybe in parser.go?), otherwise I don't think it'll work with Godep.

bwester commented 7 years ago

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 peg. Checking in generated source code will work well enough.

That Travis CI build issue you mentioned is from 2015. How has this project not run into this problem before?

DanielNi commented 7 years ago

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.