From 43c2fcc1efb9d5d98de8c828084d24edd97c10c5 Mon Sep 17 00:00:00 2001
From: Yong Tang <yongtang@users.noreply.github.com>
Date: Wed, 22 Feb 2017 10:28:04 -0800
Subject: [PATCH] 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`.
---
 Makefile | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/Makefile b/Makefile
index fc8758d5f..ecd690786 100644
--- a/Makefile
+++ b/Makefile
@@ -20,7 +20,7 @@ docker: deps
 	docker tag $(DOCKER_IMAGE_NAME):latest $(DOCKER_IMAGE_NAME):$(DOCKER_VERSION)
 
 .PHONY: deps
-deps:
+deps: fmt
 	go get ${BUILD_VERBOSE}
 
 .PHONY: test
@@ -55,6 +55,12 @@ core/zmiddleware.go core/dnsserver/zdirectives.go: middleware.cfg
 gen:
 	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
 distclean: clean
 	# Clean all dependencies and build artifacts