From 3eae75b1b892675b18aee8623f9b40ed47b949d8 Mon Sep 17 00:00:00 2001 From: "M. J. Fromberger" Date: Tue, 31 Jan 2023 11:16:39 -0800 Subject: [PATCH] ci: make gofmt check fail for a non-empty diff (#7131) Fixes #7130. Change-Id: If47eb472ea98a8d8b250c5c681c7862d252645fb Signed-off-by: M. J. Fromberger --- .github/workflows/static-analysis.yml | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/.github/workflows/static-analysis.yml b/.github/workflows/static-analysis.yml index db89c74e9..142c49313 100644 --- a/.github/workflows/static-analysis.yml +++ b/.github/workflows/static-analysis.yml @@ -24,7 +24,9 @@ jobs: with: go-version-file: go.mod - name: Run gofmt (goimports) - run: go run golang.org/x/tools/cmd/goimports -d --format-only . + run: | + OUT=$(go run golang.org/x/tools/cmd/goimports -d --format-only .) + [ -z "$OUT" ] || (echo "Not gofmt'ed: $OUT" && exit 1) - uses: k0kubun/action-slack@v2.0.0 with: payload: |