diff options
| author | Joseph Richey <joerichey94@gmail.com> | 2017-08-29 22:29:33 -0700 |
|---|---|---|
| committer | Joseph Richey <joerichey94@gmail.com> | 2017-08-29 22:29:33 -0700 |
| commit | 8539c4c12ebdd6441f5d93fd499e31df2ed943c3 (patch) | |
| tree | e0c80616666a3b9ca32a5d6869a7c01b90336063 | |
| parent | 83d4b499c505d3f5841fc0b0f8f29509622e870b (diff) | |
Go formatter "gofmt" -> "goimports"
| -rw-r--r-- | CONTRIBUTING.md | 9 | ||||
| -rw-r--r-- | Makefile | 5 |
2 files changed, 8 insertions, 6 deletions
diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index e7a2a75..20fb884 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -86,10 +86,11 @@ make test-teardown ### Formatting and Linting -The `make format` command formats all the code in fscrypt with either `gofmt` -(for Go code) or [`clang-format`](https://clang.llvm.org/docs/ClangFormat.html) -(for C code). `gofmt` comes with any Go distribution, and `clang-format` can be -installed with your package manager. +The `make format` command formats all the code in fscrypt with either +[`goimports`](https://godoc.org/golang.org/x/tools/cmd/goimports) (for Go code) +or [`clang-format`](https://clang.llvm.org/docs/ClangFormat.html) (for C code). +`goimports` can be installed with `go get`; `clang-format` can be installed +with your package manager. The `make lint` command runs a series of static analysis checks on your code. This requires the @@ -129,11 +129,11 @@ update: # Format all the Go and C code .PHONY: format format-check format: - @gofmt -l -s -w $(GO_FILES) + @goimports -l -w $(GO_FILES) @clang-format -i -style=Google $(C_FILES) format-check: - @gofmt -s -d $(GO_FILES) \ + @goimports -d $(GO_FILES) \ | ./input_fail.py "Incorrectly formatted Go files. Run \"make format\"." @clang-format -i -style=Google -output-replacements-xml $(C_FILES) \ | grep "<replacement " \ @@ -169,6 +169,7 @@ go-tools: go get -u github.com/golang/protobuf/protoc-gen-go go get -u github.com/golang/lint/golint go get -u github.com/kardianos/govendor + go get -u golang.org/x/tools/cmd/goimports go get -u honnef.co/go/tools/cmd/megacheck ##### Setup/Teardown for integration tests (need root permissions) ##### |