Browse Source

chore(ci): fail lint ci when gofmt or tidy check failed (#3085)

cong 2 years ago
parent
commit
6a692453dc
1 changed files with 11 additions and 2 deletions
  1. 11 2
      .github/workflows/go.yml

+ 11 - 2
.github/workflows/go.yml

@@ -29,8 +29,17 @@ jobs:
       - name: Lint
       - name: Lint
         run: |
         run: |
           go vet -stdmethods=false $(go list ./...)
           go vet -stdmethods=false $(go list ./...)
-          go install mvdan.cc/gofumpt@latest
-          test -z "$(gofumpt -l -extra .)" || echo "Please run 'gofumpt -l -w -extra .'"
+
+          if ! test -z "$(gofmt -l .)"; then
+            echo "Please run 'gofmt -l -w .'"
+            exit 1
+          fi
+
+          go mod tidy
+          if ! test -z "$(git status --porcelain)"; then
+            echo "Please run 'go mod tidy'"
+            exit 1
+          fi
 
 
       - name: Test
       - name: Test
         run: go test -race -coverprofile=coverage.txt -covermode=atomic ./...
         run: go test -race -coverprofile=coverage.txt -covermode=atomic ./...