Skip to content

Commit

Permalink
Improve the test workflow
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
garyburd committed May 14, 2024
1 parent 9b44fee commit e94cce5
Showing 1 changed file with 1 addition and 5 deletions.
6 changes: 1 addition & 5 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -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:
Expand Down

0 comments on commit e94cce5

Please sign in to comment.