khenaidoo | 106c61a | 2021-08-11 18:05:46 -0400 | [diff] [blame] | 1 | export GOBIN ?= $(shell pwd)/bin |
William Kurkian | ea86948 | 2019-04-09 15:16:11 -0400 | [diff] [blame] | 2 | |
khenaidoo | 106c61a | 2021-08-11 18:05:46 -0400 | [diff] [blame] | 3 | GOLINT = $(GOBIN)/golint |
| 4 | STATICCHECK = $(GOBIN)/staticcheck |
William Kurkian | ea86948 | 2019-04-09 15:16:11 -0400 | [diff] [blame] | 5 | BENCH_FLAGS ?= -cpuprofile=cpu.pprof -memprofile=mem.pprof -benchmem |
khenaidoo | 106c61a | 2021-08-11 18:05:46 -0400 | [diff] [blame] | 6 | |
| 7 | # Directories containing independent Go modules. |
| 8 | # |
| 9 | # We track coverage only for the main module. |
| 10 | MODULE_DIRS = . ./benchmarks ./zapgrpc/internal/test |
| 11 | |
William Kurkian | ea86948 | 2019-04-09 15:16:11 -0400 | [diff] [blame] | 12 | # Many Go tools take file globs or directories as arguments instead of packages. |
khenaidoo | 106c61a | 2021-08-11 18:05:46 -0400 | [diff] [blame] | 13 | GO_FILES := $(shell \ |
| 14 | find . '(' -path '*/.*' -o -path './vendor' ')' -prune \ |
| 15 | -o -name '*.go' -print | cut -b3-) |
William Kurkian | ea86948 | 2019-04-09 15:16:11 -0400 | [diff] [blame] | 16 | |
| 17 | .PHONY: all |
| 18 | all: lint test |
| 19 | |
William Kurkian | ea86948 | 2019-04-09 15:16:11 -0400 | [diff] [blame] | 20 | .PHONY: lint |
khenaidoo | 106c61a | 2021-08-11 18:05:46 -0400 | [diff] [blame] | 21 | lint: $(GOLINT) $(STATICCHECK) |
William Kurkian | ea86948 | 2019-04-09 15:16:11 -0400 | [diff] [blame] | 22 | @rm -rf lint.log |
| 23 | @echo "Checking formatting..." |
khenaidoo | 106c61a | 2021-08-11 18:05:46 -0400 | [diff] [blame] | 24 | @gofmt -d -s $(GO_FILES) 2>&1 | tee lint.log |
William Kurkian | ea86948 | 2019-04-09 15:16:11 -0400 | [diff] [blame] | 25 | @echo "Checking vet..." |
khenaidoo | 106c61a | 2021-08-11 18:05:46 -0400 | [diff] [blame] | 26 | @$(foreach dir,$(MODULE_DIRS),(cd $(dir) && go vet ./... 2>&1) &&) true | tee -a lint.log |
William Kurkian | ea86948 | 2019-04-09 15:16:11 -0400 | [diff] [blame] | 27 | @echo "Checking lint..." |
khenaidoo | 106c61a | 2021-08-11 18:05:46 -0400 | [diff] [blame] | 28 | @$(foreach dir,$(MODULE_DIRS),(cd $(dir) && $(GOLINT) ./... 2>&1) &&) true | tee -a lint.log |
| 29 | @echo "Checking staticcheck..." |
| 30 | @$(foreach dir,$(MODULE_DIRS),(cd $(dir) && $(STATICCHECK) ./... 2>&1) &&) true | tee -a lint.log |
William Kurkian | ea86948 | 2019-04-09 15:16:11 -0400 | [diff] [blame] | 31 | @echo "Checking for unresolved FIXMEs..." |
khenaidoo | 106c61a | 2021-08-11 18:05:46 -0400 | [diff] [blame] | 32 | @git grep -i fixme | grep -v -e Makefile | tee -a lint.log |
William Kurkian | ea86948 | 2019-04-09 15:16:11 -0400 | [diff] [blame] | 33 | @echo "Checking for license headers..." |
khenaidoo | 106c61a | 2021-08-11 18:05:46 -0400 | [diff] [blame] | 34 | @./checklicense.sh | tee -a lint.log |
William Kurkian | ea86948 | 2019-04-09 15:16:11 -0400 | [diff] [blame] | 35 | @[ ! -s lint.log ] |
khenaidoo | 106c61a | 2021-08-11 18:05:46 -0400 | [diff] [blame] | 36 | @echo "Checking 'go mod tidy'..." |
| 37 | @make tidy |
| 38 | @if ! git diff --quiet; then \ |
| 39 | echo "'go mod tidy' resulted in changes or working tree is dirty:"; \ |
| 40 | git --no-pager diff; \ |
| 41 | fi |
| 42 | |
| 43 | $(GOLINT): |
| 44 | cd tools && go install golang.org/x/lint/golint |
| 45 | |
| 46 | $(STATICCHECK): |
| 47 | cd tools && go install honnef.co/go/tools/cmd/staticcheck |
William Kurkian | ea86948 | 2019-04-09 15:16:11 -0400 | [diff] [blame] | 48 | |
| 49 | .PHONY: test |
| 50 | test: |
khenaidoo | 106c61a | 2021-08-11 18:05:46 -0400 | [diff] [blame] | 51 | @$(foreach dir,$(MODULE_DIRS),(cd $(dir) && go test -race ./...) &&) true |
William Kurkian | ea86948 | 2019-04-09 15:16:11 -0400 | [diff] [blame] | 52 | |
| 53 | .PHONY: cover |
| 54 | cover: |
khenaidoo | 106c61a | 2021-08-11 18:05:46 -0400 | [diff] [blame] | 55 | go test -race -coverprofile=cover.out -coverpkg=./... ./... |
| 56 | go tool cover -html=cover.out -o cover.html |
William Kurkian | ea86948 | 2019-04-09 15:16:11 -0400 | [diff] [blame] | 57 | |
| 58 | .PHONY: bench |
| 59 | BENCH ?= . |
| 60 | bench: |
khenaidoo | 106c61a | 2021-08-11 18:05:46 -0400 | [diff] [blame] | 61 | @$(foreach dir,$(MODULE_DIRS), ( \ |
| 62 | cd $(dir) && \ |
| 63 | go list ./... | xargs -n1 go test -bench=$(BENCH) -run="^$$" $(BENCH_FLAGS) \ |
| 64 | ) &&) true |
William Kurkian | ea86948 | 2019-04-09 15:16:11 -0400 | [diff] [blame] | 65 | |
| 66 | .PHONY: updatereadme |
| 67 | updatereadme: |
| 68 | rm -f README.md |
| 69 | cat .readme.tmpl | go run internal/readme/readme.go > README.md |
khenaidoo | 106c61a | 2021-08-11 18:05:46 -0400 | [diff] [blame] | 70 | |
| 71 | .PHONY: tidy |
| 72 | tidy: |
| 73 | @$(foreach dir,$(MODULE_DIRS),(cd $(dir) && go mod tidy) &&) true |