-
Notifications
You must be signed in to change notification settings - Fork 20
Conversation
026bd22
to
9dc75f9
Compare
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.
Just a couple nits, other than that lgtm
Once we get some Go code in here, please setup travis https://github.com/coreos/matchbox/blob/master/.travis.yml
Makefile
Outdated
|
||
check: | ||
@gofmt -l -s $(GOFILES) | read; if [ $$? == 0 ]; then gofmt -s -d $(GOFILES); exit 1; fi | ||
@go vet $(shell go list ./... | grep -v '/vendor/') |
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.
nit: go list ./...
doesn't match vendor anymore https://golang.org/doc/go1.9#vendor-dotdotdot
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.
fixed this
Makefile
Outdated
@gofmt -l -s $(GOFILES) | read; if [ $$? == 0 ]; then gofmt -s -d $(GOFILES); exit 1; fi | ||
@go vet $(shell go list ./... | grep -v '/vendor/') | ||
@./scripts/verify-gopkg.sh | ||
@go test -v $(shell go list ./... | grep -v '/vendor/\|/e2e') |
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.
Can you add golint
?
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.
added golint
9dc75f9
to
b59676a
Compare
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. We'll see if it all works when we setup Travis :)
OWNERS: init
/cc: @ericchiang