Skip to content

Commit c542edf

Browse files
committed
Use golangci-lint to run govet
1 parent c409cd9 commit c542edf

File tree

4 files changed

+3
-10
lines changed

4 files changed

+3
-10
lines changed

.golangci.toml

+1
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ format = "colored-line-number"
88
disable-all = true
99
enable = [
1010
"gofumpt",
11+
"govet",
1112
]
1213

1314
[issues]

circle.yml

-8
Original file line numberDiff line numberDiff line change
@@ -72,14 +72,6 @@ jobs:
7272
executor: gopherjs
7373
steps:
7474
- setup_and_install_gopherjs
75-
- run:
76-
name: Check go vet
77-
command: |
78-
set -e
79-
# Go package in root directory.
80-
go vet .
81-
# All subdirectories except "doc", "tests", "node*".
82-
for d in */; do echo ./$d...; done | grep -v ./doc | grep -v ./tests | grep -v ./node | xargs go vet
8375
- run:
8476
name: Check natives build tags
8577
command: diff -u <(echo -n) <(go list ./compiler/natives/src/...) # All those packages should have // +build js.

tests/goroutine_test.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ func testPanic2(t *testing.T) {
7373
time.Sleep(0)
7474
checkI(t, 4)
7575
panic(7)
76-
checkI(t, -3)
76+
checkI(t, -3) //nolint:govet // Unreachable code is intentional for panic test
7777
}
7878

7979
func TestPanicAdvanced(t *testing.T) {

tests/misc_test.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -536,7 +536,7 @@ func TestTrivialSwitch(t *testing.T) {
536536
}
537537
return
538538
}
539-
t.Fail()
539+
t.Fail() //nolint:govet // unreachable code intentional for test
540540
}
541541

542542
func TestTupleFnReturnImplicitCast(t *testing.T) {

0 commit comments

Comments
 (0)