alexlebens/external-snapshotter
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

build.make: more readable "make test" output

Browse Source 
The introduction for each individual test looked like an actual
command:

  test-subtree
  ./release-tools/verify-subtree.sh release-tools
  Directory 'release-tools' contains non-upstream changes:
  ...

It's better to make it look like a shell comment and increase its
visibility with a longer prefix:

  ### test-subtree:
  ./release-tools/verify-subtree.sh release-tools
  ...
This commit is contained in:
Patrick Ohly
2019-01-25 11:53:36 +01:00
parent 147892c954
commit b0336b553c
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
build.make
View File

@@ -97,19 +97,19 @@ test:
.PHONY: test-go .PHONY: test-go
test: test-go test: test-go
test-go: test-go:
@ echo; echo $@ @ echo; echo "### $@:"
go test `go list ./... | grep -v 'vendor' $(TEST_GO_FILTER_CMD)` $(TESTARGS) go test `go list ./... | grep -v 'vendor' $(TEST_GO_FILTER_CMD)` $(TESTARGS)
.PHONY: test-vet .PHONY: test-vet
test: test-vet test: test-vet
test-vet: test-vet:
@ echo; echo $@ @ echo; echo "### $@:"
go vet `go list ./... | grep -v vendor $(TEST_VET_FILTER_CMD)` go vet `go list ./... | grep -v vendor $(TEST_VET_FILTER_CMD)`
.PHONY: test-fmt .PHONY: test-fmt
test: test-fmt test: test-fmt
test-fmt: test-fmt:
@ echo; echo $@ @ echo; echo "### $@:"
files=$$(find . -name '*.go' | grep -v './vendor' $(TEST_FMT_FILTER_CMD)); \ files=$$(find . -name '*.go' | grep -v './vendor' $(TEST_FMT_FILTER_CMD)); \
if [ $$(gofmt -d $$files | wc -l) -ne 0 ]; then \ if [ $$(gofmt -d $$files | wc -l) -ne 0 ]; then \
echo "formatting errors:"; \ echo "formatting errors:"; \
@@ -120,5 +120,5 @@ test-fmt:
.PHONY: test-subtree .PHONY: test-subtree
test: test-subtree test: test-subtree
test-subtree: test-subtree:
@ echo; echo $@ @ echo; echo "### $@:"
./release-tools/verify-subtree.sh release-tools ./release-tools/verify-subtree.sh release-tools