From 01c2d728ea7cc42fa3cbaa140dc4ecc7166b13ac Mon Sep 17 00:00:00 2001 From: Gary Burd Date: Mon, 13 May 2024 13:16:55 -0700 Subject: [PATCH] Improve the test workflow Remove the go vet step. Because the go test command runs go vet, there's no need to run go vet separately. Remove the verbose flag from the go test step. The verbose output provides little value and gets in the way of seeing the actual problems. --- .github/workflows/test.yml | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 8ea3574e..299fb016 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -55,13 +55,9 @@ jobs: neovim: true version: ${{ matrix.neovim-version }} - - name: go vet - run: | - go vet ./... - - name: Test and take a coverage run: | - go test -v -race -count=1 -covermode=atomic -coverpkg=./... -coverprofile=coverage.out ./... + go test -race -count=1 -covermode=atomic -coverpkg=./... -coverprofile=coverage.out ./... - uses: codecov/codecov-action@v3 with: