Skip to content

Commit efdb0c5

Browse files
committed
build(lint): Linter version bump to v2.2.1 and configuration migration for linter
1 parent 1eb6d8b commit efdb0c5

File tree

2 files changed

+69
-60
lines changed

2 files changed

+69
-60
lines changed

.golangci.yml

Lines changed: 68 additions & 59 deletions
Original file line numberDiff line numberDiff line change
@@ -1,58 +1,6 @@
1-
linters-settings:
2-
dupl:
3-
threshold: 100
4-
depguard:
5-
rules:
6-
Test:
7-
files:
8-
- $test
9-
allow:
10-
- $gostd
11-
- github.com/stretchr/testify
12-
gci:
13-
sections:
14-
- standard # Standard section: captures all standard packages.
15-
- default
16-
- prefix(alexejk.io/go-xmlrpc)
17-
goconst:
18-
min-len: 2
19-
min-occurrences: 2
20-
gocritic:
21-
enabled-tags:
22-
- diagnostic
23-
- experimental
24-
- opinionated
25-
- performance
26-
- style
27-
disabled-checks:
28-
- dupImport # https://github.com/go-critic/go-critic/issues/845
29-
- ifElseChain
30-
- octalLiteral
31-
- whyNoLint
32-
- wrapperFunc
33-
goimports:
34-
local-prefixes: alexejk.io/go-xmlrpc
35-
mnd:
36-
settings:
37-
mnd:
38-
# don't include the "operation" and "assign"
39-
checks: argument,case,condition,return
40-
govet:
41-
shadow: false
42-
exhaustive:
43-
default-signifies-exhaustive: true
44-
maligned:
45-
suggest-new: true
46-
misspell:
47-
locale: US
48-
nolintlint:
49-
allow-leading-space: true # don't require machine-readable nolint directives (i.e. with no leading space)
50-
allow-unused: false # report any unused nolint directives
51-
require-explanation: false # don't require an explanation for nolint directives
52-
require-specific: false # don't require nolint directives to be specific about which linter is being skipped
53-
1+
version: "2"
542
linters:
55-
disable-all: true
3+
default: none
564
enable:
575
- bodyclose
586
- depguard
@@ -63,22 +11,83 @@ linters:
6311
- gochecknoinits
6412
- goconst
6513
- gocritic
66-
- gofmt
67-
- goimports
68-
- mnd
6914
- goprintffuncname
7015
- gosec
71-
- gosimple
7216
- govet
7317
- ineffassign
7418
- misspell
19+
- mnd
7520
- nakedret
7621
- noctx
7722
- nolintlint
7823
- rowserrcheck
7924
- staticcheck
80-
- typecheck
8125
- unconvert
8226
- unparam
8327
- unused
8428
- whitespace
29+
settings:
30+
depguard:
31+
rules:
32+
Test:
33+
files:
34+
- $test
35+
allow:
36+
- $gostd
37+
- github.com/stretchr/testify
38+
dupl:
39+
threshold: 100
40+
exhaustive:
41+
default-signifies-exhaustive: true
42+
goconst:
43+
min-len: 2
44+
min-occurrences: 2
45+
gocritic:
46+
disabled-checks:
47+
- dupImport
48+
- ifElseChain
49+
- octalLiteral
50+
- whyNoLint
51+
- wrapperFunc
52+
enabled-tags:
53+
- diagnostic
54+
- experimental
55+
- opinionated
56+
- performance
57+
- style
58+
misspell:
59+
locale: US
60+
nolintlint:
61+
require-explanation: true
62+
require-specific: false
63+
allow-unused: false
64+
exclusions:
65+
generated: lax
66+
presets:
67+
- comments
68+
- common-false-positives
69+
- legacy
70+
- std-error-handling
71+
paths:
72+
- third_party$
73+
- builtin$
74+
- examples$
75+
formatters:
76+
enable:
77+
- gofmt
78+
- goimports
79+
settings:
80+
gci:
81+
sections:
82+
- standard
83+
- default
84+
- prefix(alexejk.io/go-xmlrpc)
85+
goimports:
86+
local-prefixes:
87+
- alexejk.io/go-xmlrpc
88+
exclusions:
89+
generated: lax
90+
paths:
91+
- third_party$
92+
- builtin$
93+
- examples$

hack/linter.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ GREEN="\033[32m"
66
YELLOW="\033[33m"
77
NORMAL="\033[39m"
88

9-
LINTER_VERSION=1.64.5
9+
LINTER_VERSION=2.2.1
1010

1111
LINTER_BINDIR=$(go env GOPATH)/bin
1212
LINTER_NAME=golangci-lint

0 commit comments

Comments
 (0)