[WIP] Add Go tool
Review Request #10302 — Created Nov. 1, 2018 and updated
Add Go tool
Add more support for the Go language. This patch adds a Go Tool class which
runs two commands on go files:
1) go tool vet # static code analysis to find possible bugs or suspicious
2) gofmt # formats the code
Revision 2 (+92)
We might want to make a note that we are using
go fmtbecause the former does not allow us to specify options that we want to use.
In the case of a string that contains single quotes, it is more readable to use double quotes, e.g.
"%s's formatting differs from gofmt's"
I think this is a typo because
line.split(':', 1)will always two values if
:is in line. Should this say
If split does not return two values?
Also this should be in the except block.
Comments must be proper sentences.