redspread / spread

Docker to Kubernetes in one command
https://coreos.com
Apache License 2.0
1.84k stars 83 forks source link

gofmt -s -w #134

Closed zoidyzoidzoid closed 8 years ago

zoidyzoidzoid commented 8 years ago

make wouldn't run successfully without this file being formatted.

I'm not sure how the previous travis run passed, because when I run make validate, I get:

± make validate
for pkg in ./pkg/... ./cli/... ./cmd/...; do \
        echo "Running golint on $i:"; \
        golint $i; \
    done;
Running golint on :
Running golint on :
Running golint on :
# get all go files and run go fmt on them
files=$(find . -name '*.go' -not -path "./vendor/*" | xargs gofmt  -l); if [[ -n "$files" ]]; then \
          echo "Error: 'gofmt ' needs to be run on:"; \
          echo "${files}"; \
          exit 1; \
          fi;
Error: 'gofmt ' needs to be run on:
./cli/cluster.go
make: *** [checkgofmt] Error 1
ethernetdan commented 8 years ago

LGTM - opened an issue (#135) on the builds passing