Commit 43c2fcc1 authored by Yong Tang's avatar Yong Tang Committed by John Belamaric

Add dependency of `go fmt` in `Makefile` (#553)

This fix adds the dependency of `go fmt` in `Makefile`
so that any new check in will have to properly fmt the source code
with `gofmt -s -w`.
parent 4723fb94
...@@ -20,7 +20,7 @@ docker: deps ...@@ -20,7 +20,7 @@ docker: deps
docker tag $(DOCKER_IMAGE_NAME):latest $(DOCKER_IMAGE_NAME):$(DOCKER_VERSION) docker tag $(DOCKER_IMAGE_NAME):latest $(DOCKER_IMAGE_NAME):$(DOCKER_VERSION)
.PHONY: deps .PHONY: deps
deps: deps: fmt
go get ${BUILD_VERBOSE} go get ${BUILD_VERBOSE}
.PHONY: test .PHONY: test
...@@ -55,6 +55,12 @@ core/zmiddleware.go core/dnsserver/zdirectives.go: middleware.cfg ...@@ -55,6 +55,12 @@ core/zmiddleware.go core/dnsserver/zdirectives.go: middleware.cfg
gen: gen:
go generate coredns.go go generate coredns.go
.PHONY: fmt
fmt:
## run go fmt
@test -z "$$(gofmt -s -l . | grep -v vendor/ | tee /dev/stderr)" || \
(echo "please format Go code with 'gofmt -s -w'" && false)
.PHONY: distclean .PHONY: distclean
distclean: clean distclean: clean
# Clean all dependencies and build artifacts # Clean all dependencies and build artifacts
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment