search

FactomProject / factomd

Factom Daemon
https://www.factomprotocol.org/
Other
201 stars 92 forks source link

Fix gofmt step for circleci #1055

Closed PaulBernier closed 4 years ago

PaulBernier commented 4 years ago

This PR does 3 things:

  1. Make the gofmt step actually work in CircleCI: until know that step would always work no matter what is pushed, because it was using option -w that just rewrites files locally...
  2. But gofmt step in front of tests, we should fail because of linting as early as possible, no need to waste time testing if the job is going to fail anyway.
  3. Fix one file that was not properly formatted (but because of 1. never got noticed)