forked from bojand/ghz
-
Notifications
You must be signed in to change notification settings - Fork 0
/
.golangci.yml
41 lines (34 loc) · 1.17 KB
/
.golangci.yml
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
# TODO Look into enabling all with some exception where practical
# linters:
# enable-all: true
# disable:
# - gochecknoglobals
# - dupl
# - lll
issues:
exclude-rules:
# Exclude lostcancel govet rule specifically for requester.go
# Since we purposefully do that. See comments in code.
- path: runner/requester.go
text: "lostcancel"
# TODO Look into fixing time.Tick() usage SA1015 in worker.go
- path: runner/worker.go
text: "SA1015"
# We intentionally assign nil to err
- path: runner/worker.go
text: "ineffectual assignment to `err`"
# Debug log Sync() error check in defer in main
- path: cmd/ghz/main.go
text: "Error return value of `logger.Sync` is not checked"
# sync.once copy in pacer test
- path: load/pacer_test.go
text: "copylocks"
# TODO fix protobuf deprecated
- path: runner/
text: 'SA1019: "github.com/golang/protobuf/proto"'
# TODO fix protobuf deprecated
- path: protodesc/
text: 'SA1019: "github.com/golang/protobuf/proto"'
# TODO fix protobuf deprecated
- path: runner/
text: 'SA1019: "github.com/golang/protobuf/jsonpb" is deprecated'