Skip to content

Commit 75b08c1

Browse files
committed
fix lint
Signed-off-by: Michael Crenshaw <[email protected]>
1 parent 49b65f1 commit 75b08c1

1 file changed

Lines changed: 9 additions & 14 deletions

File tree

.golangci.yaml

Lines changed: 9 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,8 @@ issues:
1313

1414
max-same-issues: 0
1515

16+
# Some linters are disabled on this branch to keep the go 1.25 -> 1.26 upgrade minimal.
17+
# More linters are enabled on master.
1618
linters:
1719
enable:
1820
- errorlint
@@ -22,24 +24,18 @@ linters:
2224
- govet
2325
- importas
2426
- misspell
25-
- modernize
26-
- noctx
2727
- perfsprint
2828
- revive
2929
- staticcheck
3030
- testifylint
3131
- thelper
3232
- tparallel
33-
- unparam
3433
- usestdlibvars
3534
- usetesting
3635
- whitespace
3736

3837
exclusions:
39-
rules:
40-
- linters:
41-
- unparam
42-
path: (.+)_test\.go
38+
rules: []
4339

4440
presets:
4541
- comments
@@ -59,6 +55,7 @@ linters:
5955
- assignOp # Leave disabled, assign operations can be more confusing than helpful.
6056
- commentedOutCode
6157
- deferInLoop
58+
- deprecatedComment
6259
- exitAfterDefer
6360
- hugeParam
6461
- importShadow
@@ -122,13 +119,6 @@ linters:
122119
- pkg: github.com/argoproj/argo-cd/v3/util/io
123120
alias: utilio
124121

125-
modernize:
126-
disable:
127-
# Suggest replacing omitempty with omitzero for struct fields.
128-
- omitzero
129-
# Simplify code by using go1.26's new(expr). - generates lots of false positives.
130-
- newexpr
131-
132122
nolintlint:
133123
require-specific: true
134124

@@ -143,6 +133,8 @@ linters:
143133
sprintf1: true
144134
# Optimizes into strings concatenation.
145135
strconcat: true
136+
# Re-enable once master picks up perfsprint cleanups (concat in loop).
137+
concat-loop: false
146138

147139
revive:
148140
enable-all-rules: false
@@ -219,7 +211,9 @@ linters:
219211
- name: unexported-return
220212
disabled: true
221213

214+
# Re-enable once master updates printf-style CLI output.
222215
- name: unnecessary-format
216+
disabled: true
223217

224218
- name: unnecessary-stmt
225219

@@ -243,6 +237,7 @@ linters:
243237
staticcheck:
244238
checks:
245239
- all
240+
- -SA5008
246241
- -SA5011
247242
- -ST1003
248243
- -ST1016

0 commit comments

Comments
 (0)