Consider formatting code with `gofmt`

[bbrowning]

Many Golang project use gofmt to consistently format their Go code. And, it's not uncommon to setup your editor to automatically run gofmt when saving changes to a file. Perhaps we should consider doing so as well?

[bbrowning]

Here's an example of how gofmt would change the existing code - bbrowning@076935a

It's mainly consistent whitespace and import ordering changes.

[jstrachan]

Good idea! We should probably hack the Makefile to invoke it - maybe on the test target?

[jimmidyson]

Most projects set up CI validation for this.

[maleck13]

We use the following for this in our makefile. Happy to make a PR

check-gofmt:
	diff <(gofmt -d -s .) <(printf "")

.PHONY: check-golint
check-golint:
	diff <(golint ./... | grep -v vendor/) <(printf "")

[jstrachan]

@maleck13 that'd be awesome if you could thanks!