go fmt and other things

[jbenet]

I want to make sure all code pushed is run through go fmt. We probably want:

• a git pre commit hook
• a github integration that checks PRs (go fmt, go test)

This post (http://peter.bourgon.org/go-in-production/) also recommends:

on Save: go fmt (or goimports)
on Build: go vet, golint, and maybe go test
on Deploy: go test -tags=integration

[jbenet]

cc @sqs thoughts?

[sqs]

Travis-CI is great for go test. (https://github.com/sourcegraph/srclib/blob/a672df5a0adaa762a66db579f7d13fa3aa6523e3/.travis.yml)

As for go fmt, in my experience 98% of Go contributors already enable it in their editor. But yeah, for accepting contributions, it's probably nice.

go vet/lint are not intended to be strictly adhered to in all cases, and there's no way of "ignoring" warnings, so those are probably best run manually or as optional steps.

[jbenet]

thanks @sqs ! i've added go test with Travis-CI, and go fmt to the readme. that should be good for now.