chore(ci): fix redirect with GITHUB_STEP_SUMMARY

If GITHUB_STEP_SUMMARY isn't set then the redirect should be to /dev/null, even if it isn't called the shell parser will complain
IBM · Jul 20, 2022 · e74aea4 · e74aea4
1 parent d96db86
commit e74aea4
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/Makefile b/Makefile
@@ -29,12 +29,11 @@ lint:
 test: $(GOBIN)/tparse
 	$(GOTEST) -timeout 2m -json ./... \
 		| tee output.json | $(GOBIN)/tparse -follow -all
-	[ -z "${GITHUB_STEP_SUMMARY}" ] \
-		|| NO_COLOR=1 $(GOBIN)/tparse -format markdown -file output.json -all >${GITHUB_STEP_SUMMARY}
-
+	[ -z "$${GITHUB_STEP_SUMMARY}" ] \
+		|| NO_COLOR=1 $(GOBIN)/tparse -format markdown -file output.json -all >"$${GITHUB_STEP_SUMMARY:-/dev/null}"
 .PHONY: test_functional
 test_functional: $(GOBIN)/tparse
 	$(GOTEST) -timeout 12m -tags=functional -json ./... \
 		| tee output.json | $(GOBIN)/tparse -follow -all
-	[ -z "${GITHUB_STEP_SUMMARY}" ] \
-		|| NO_COLOR=1 $(GOBIN)/tparse -format markdown -file output.json -all >${GITHUB_STEP_SUMMARY}
+	[ -z "$${GITHUB_STEP_SUMMARY:-}" ] \
+		|| NO_COLOR=1 $(GOBIN)/tparse -format markdown -file output.json -all >"$${GITHUB_STEP_SUMMARY:-/dev/null}"