forked from golangci/golangci-lint
-
Notifications
You must be signed in to change notification settings - Fork 0
/
.golangci.next.reference.yml
4177 lines (3972 loc) · 147 KB
/
.golangci.next.reference.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
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
424
425
426
427
428
429
430
431
432
433
434
435
436
437
438
439
440
441
442
443
444
445
446
447
448
449
450
451
452
453
454
455
456
457
458
459
460
461
462
463
464
465
466
467
468
469
470
471
472
473
474
475
476
477
478
479
480
481
482
483
484
485
486
487
488
489
490
491
492
493
494
495
496
497
498
499
500
501
502
503
504
505
506
507
508
509
510
511
512
513
514
515
516
517
518
519
520
521
522
523
524
525
526
527
528
529
530
531
532
533
534
535
536
537
538
539
540
541
542
543
544
545
546
547
548
549
550
551
552
553
554
555
556
557
558
559
560
561
562
563
564
565
566
567
568
569
570
571
572
573
574
575
576
577
578
579
580
581
582
583
584
585
586
587
588
589
590
591
592
593
594
595
596
597
598
599
600
601
602
603
604
605
606
607
608
609
610
611
612
613
614
615
616
617
618
619
620
621
622
623
624
625
626
627
628
629
630
631
632
633
634
635
636
637
638
639
640
641
642
643
644
645
646
647
648
649
650
651
652
653
654
655
656
657
658
659
660
661
662
663
664
665
666
667
668
669
670
671
672
673
674
675
676
677
678
679
680
681
682
683
684
685
686
687
688
689
690
691
692
693
694
695
696
697
698
699
700
701
702
703
704
705
706
707
708
709
710
711
712
713
714
715
716
717
718
719
720
721
722
723
724
725
726
727
728
729
730
731
732
733
734
735
736
737
738
739
740
741
742
743
744
745
746
747
748
749
750
751
752
753
754
755
756
757
758
759
760
761
762
763
764
765
766
767
768
769
770
771
772
773
774
775
776
777
778
779
780
781
782
783
784
785
786
787
788
789
790
791
792
793
794
795
796
797
798
799
800
801
802
803
804
805
806
807
808
809
810
811
812
813
814
815
816
817
818
819
820
821
822
823
824
825
826
827
828
829
830
831
832
833
834
835
836
837
838
839
840
841
842
843
844
845
846
847
848
849
850
851
852
853
854
855
856
857
858
859
860
861
862
863
864
865
866
867
868
869
870
871
872
873
874
875
876
877
878
879
880
881
882
883
884
885
886
887
888
889
890
891
892
893
894
895
896
897
898
899
900
901
902
903
904
905
906
907
908
909
910
911
912
913
914
915
916
917
918
919
920
921
922
923
924
925
926
927
928
929
930
931
932
933
934
935
936
937
938
939
940
941
942
943
944
945
946
947
948
949
950
951
952
953
954
955
956
957
958
959
960
961
962
963
964
965
966
967
968
969
970
971
972
973
974
975
976
977
978
979
980
981
982
983
984
985
986
987
988
989
990
991
992
993
994
995
996
997
998
999
1000
# This file contains all available configuration options
# with their default values (in comments).
#
# This file is not a configuration example,
# it contains the exhaustive configuration with explanations of the options.
linters:
# Disable all linters.
# Default: false
disable-all: true
# Enable specific linter
# https://golangci-lint.run/usage/linters/#enabled-by-default
enable:
- asasalint
- asciicheck
- bidichk
- bodyclose
- canonicalheader
- containedctx
- contextcheck
- copyloopvar
- cyclop
- decorder
- depguard
- dogsled
- dupl
- dupword
- durationcheck
- err113
- errcheck
- errchkjson
- errname
- errorlint
- execinquery
- exhaustive
- exhaustruct
- exportloopref
- fatcontext
- forbidigo
- forcetypeassert
- funlen
- gci
- ginkgolinter
- gocheckcompilerdirectives
- gochecknoglobals
- gochecknoinits
- gochecksumtype
- gocognit
- goconst
- gocritic
- gocyclo
- godot
- godox
- gofmt
- gofumpt
- goheader
- goimports
- gomoddirectives
- gomodguard
- goprintffuncname
- gosec
- gosimple
- gosmopolitan
- govet
- grouper
- iface
- importas
- inamedparam
- ineffassign
- interfacebloat
- intrange
- ireturn
- lll
- loggercheck
- maintidx
- makezero
- mirror
- misspell
- mnd
- musttag
- nakedret
- nestif
- nilerr
- nilnil
- nlreturn
- noctx
- nolintlint
- nonamedreturns
- nosprintfhostport
- paralleltest
- perfsprint
- prealloc
- predeclared
- promlinter
- protogetter
- reassign
- recvcheck
- revive
- rowserrcheck
- sloglint
- spancheck
- sqlclosecheck
- staticcheck
- stylecheck
- tagalign
- tagliatelle
- tenv
- testableexamples
- testifylint
- testpackage
- thelper
- tparallel
- unconvert
- unparam
- unused
- usestdlibvars
- usetesting
- varnamelen
- wastedassign
- whitespace
- wrapcheck
- wsl
- zerologlint
# Enable all available linters.
# Default: false
enable-all: true
# Disable specific linter
# https://golangci-lint.run/usage/linters/#disabled-by-default
disable:
- asasalint
- asciicheck
- bidichk
- bodyclose
- canonicalheader
- containedctx
- contextcheck
- copyloopvar
- cyclop
- decorder
- depguard
- dogsled
- dupl
- dupword
- durationcheck
- err113
- errcheck
- errchkjson
- errname
- errorlint
- execinquery
- exhaustive
- exhaustruct
- exportloopref
- fatcontext
- forbidigo
- forcetypeassert
- funlen
- gci
- ginkgolinter
- gocheckcompilerdirectives
- gochecknoglobals
- gochecknoinits
- gochecksumtype
- gocognit
- goconst
- gocritic
- gocyclo
- godot
- godox
- gofmt
- gofumpt
- goheader
- goimports
- gomoddirectives
- gomodguard
- goprintffuncname
- gosec
- gosimple
- gosmopolitan
- govet
- grouper
- iface
- importas
- inamedparam
- ineffassign
- interfacebloat
- intrange
- ireturn
- lll
- loggercheck
- maintidx
- makezero
- mirror
- misspell
- mnd
- musttag
- nakedret
- nestif
- nilerr
- nilnil
- nlreturn
- noctx
- nolintlint
- nonamedreturns
- nosprintfhostport
- paralleltest
- perfsprint
- prealloc
- predeclared
- promlinter
- protogetter
- reassign
- recvcheck
- revive
- rowserrcheck
- sloglint
- spancheck
- sqlclosecheck
- staticcheck
- stylecheck
- tagalign
- tagliatelle
- tenv
- testableexamples
- testifylint
- testpackage
- thelper
- tparallel
- unconvert
- unparam
- unused
- usestdlibvars
- usetesting
- varnamelen
- wastedassign
- whitespace
- wrapcheck
- wsl
- zerologlint
- deadcode # Deprecated
- exhaustivestruct # Deprecated
- golint # Deprecated
- ifshort # Deprecated
- interfacer # Deprecated
- maligned # Deprecated
- gomnd # Deprecated
- nosnakecase # Deprecated
- scopelint # Deprecated
- structcheck # Deprecated
- varcheck # Deprecated
# Enable presets.
# https://golangci-lint.run/usage/linters
# Default: []
presets:
- bugs
- comment
- complexity
- error
- format
- import
- metalinter
- module
- performance
- sql
- style
- test
- unused
# Enable only fast linters from enabled linters set (first run won't be fast)
# Default: false
fast: true
# All available settings of specific linters.
linters-settings:
asasalint:
# To specify a set of function names to exclude.
# The values are merged with the builtin exclusions.
# The builtin exclusions can be disabled by setting `use-builtin-exclusions` to `false`.
# Default: ["^(fmt|log|logger|t|)\.(Print|Fprint|Sprint|Fatal|Panic|Error|Warn|Warning|Info|Debug|Log)(|f|ln)$"]
exclude:
- Append
- \.Wrapf
# To enable/disable the asasalint builtin exclusions of function names.
# See the default value of `exclude` to get the builtin exclusions.
# Default: true
use-builtin-exclusions: false
# Ignore *_test.go files.
# Default: false
ignore-test: true
bidichk:
# The following configurations check for all mentioned invisible Unicode runes.
# All runes are enabled by default.
left-to-right-embedding: false
right-to-left-embedding: false
pop-directional-formatting: false
left-to-right-override: false
right-to-left-override: false
left-to-right-isolate: false
right-to-left-isolate: false
first-strong-isolate: false
pop-directional-isolate: false
copyloopvar:
# Check all assigning the loop variable to another variable.
# Default: false
check-alias: true
cyclop:
# The maximal code complexity to report.
# Default: 10
max-complexity: 10
# The maximal average package complexity.
# If it's higher than 0.0 (float) the check is enabled
# Default: 0.0
package-average: 0.5
# Should ignore tests.
# Default: false
skip-tests: true
decorder:
# Required order of `type`, `const`, `var` and `func` declarations inside a file.
# Default: types before constants before variables before functions.
dec-order:
- type
- const
- var
- func
# If true, underscore vars (vars with "_" as the name) will be ignored at all checks
# Default: false (underscore vars are not ignored)
ignore-underscore-vars: false
# If true, order of declarations is not checked at all.
# Default: true (disabled)
disable-dec-order-check: false
# If true, `init` func can be anywhere in file (does not have to be declared before all other functions).
# Default: true (disabled)
disable-init-func-first-check: false
# If true, multiple global `type`, `const` and `var` declarations are allowed.
# Default: true (disabled)
disable-dec-num-check: false
# If true, type declarations will be ignored for dec num check
# Default: false (type statements are not ignored)
disable-type-dec-num-check: false
# If true, const declarations will be ignored for dec num check
# Default: false (const statements are not ignored)
disable-const-dec-num-check: false
# If true, var declarations will be ignored for dec num check
# Default: false (var statements are not ignored)
disable-var-dec-num-check: false
depguard:
# Rules to apply.
#
# Variables:
# - File Variables
# you can still use and exclamation mark ! in front of a variable to say not to use it.
# Example !$test will match any file that is not a go test file.
#
# `$all` - matches all go files
# `$test` - matches all go test files
#
# - Package Variables
#
# `$gostd` - matches all of go's standard library (Pulled from `GOROOT`)
#
# Default: Only allow $gostd in all files.
rules:
# Name of a rule.
main:
# Used to determine the package matching priority.
# There are three different modes: `original`, `strict`, and `lax`.
# Default: "original"
list-mode: lax
# List of file globs that will match this list of settings to compare against.
# Default: $all
files:
- "!**/*_a _file.go"
# List of allowed packages.
allow:
- $gostd
- github.com/OpenPeeDeeP
# Packages that are not allowed where the value is a suggestion.
deny:
- pkg: "github.com/sirupsen/logrus"
desc: not allowed
- pkg: "github.com/pkg/errors"
desc: Should be replaced by standard lib errors package
dogsled:
# Checks assignments with too many blank identifiers.
# Default: 2
max-blank-identifiers: 3
dupl:
# Tokens count to trigger issue.
# Default: 150
threshold: 100
dupword:
# Keywords for detecting duplicate words.
# If this list is not empty, only the words defined in this list will be detected.
# Default: []
keywords:
- "the"
- "and"
- "a"
# Keywords used to ignore detection.
# Default: []
ignore:
- "0C0C"
errcheck:
# Report about not checking of errors in type assertions: `a := b.(MyStruct)`.
# Such cases aren't reported by default.
# Default: false
check-type-assertions: true
# report about assignment of errors to blank identifier: `num, _ := strconv.Atoi(numStr)`.
# Such cases aren't reported by default.
# Default: false
check-blank: true
# To disable the errcheck built-in exclude list.
# See `-excludeonly` option in https://github.com/kisielk/errcheck#excluding-functions for details.
# Default: false
disable-default-exclusions: true
# List of functions to exclude from checking, where each entry is a single function to exclude.
# See https://github.com/kisielk/errcheck#excluding-functions for details.
exclude-functions:
- io/ioutil.ReadFile
- io.Copy(*bytes.Buffer)
- io.Copy(os.Stdout)
errchkjson:
# With check-error-free-encoding set to true, errchkjson does warn about errors
# from json encoding functions that are safe to be ignored,
# because they are not possible to happen.
#
# if check-error-free-encoding is set to true and errcheck linter is enabled,
# it is recommended to add the following exceptions to prevent from false positives:
#
# linters-settings:
# errcheck:
# exclude-functions:
# - encoding/json.Marshal
# - encoding/json.MarshalIndent
#
# Default: false
check-error-free-encoding: true
# Issue on struct encoding that doesn't have exported fields.
# Default: false
report-no-exported: false
errorlint:
# Check whether fmt.Errorf uses the %w verb for formatting errors.
# See the https://github.com/polyfloyd/go-errorlint for caveats.
# Default: true
errorf: false
# Permit more than 1 %w verb, valid per Go 1.20 (Requires errorf:true)
# Default: true
errorf-multi: false
# Check for plain type assertions and type switches.
# Default: true
asserts: false
# Check for plain error comparisons.
# Default: true
comparison: false
# Allowed errors.
# Default: []
allowed-errors:
- err: "io.EOF"
fun: "example.com/pkg.Read"
# Allowed error "wildcards".
# Default: []
allowed-errors-wildcard:
- err: "example.com/pkg.ErrMagic"
fun: "example.com/pkg.Magic"
exhaustive:
# Program elements to check for exhaustiveness.
# Default: [ switch ]
check:
- switch
- map
# Check switch statements in generated files also.
# Default: false
check-generated: true
# Presence of "default" case in switch statements satisfies exhaustiveness,
# even if all enum members are not listed.
# Default: false
default-signifies-exhaustive: true
# Enum members matching the supplied regex do not have to be listed in
# switch statements to satisfy exhaustiveness.
# Default: ""
ignore-enum-members: "Example.+"
# Enum types matching the supplied regex do not have to be listed in
# switch statements to satisfy exhaustiveness.
# Default: ""
ignore-enum-types: "Example.+"
# Consider enums only in package scopes, not in inner scopes.
# Default: false
package-scope-only: true
# Only run exhaustive check on switches with "//exhaustive:enforce" comment.
# Default: false
explicit-exhaustive-switch: true
# Only run exhaustive check on map literals with "//exhaustive:enforce" comment.
# Default: false
explicit-exhaustive-map: true
# Switch statement requires default case even if exhaustive.
# Default: false
default-case-required: true
exhaustruct:
# List of regular expressions to match struct packages and their names.
# Regular expressions must match complete canonical struct package/name/structname.
# If this list is empty, all structs are tested.
# Default: []
include:
- '.+\.Test'
- 'example\.com/package\.ExampleStruct[\d]{1,2}'
# List of regular expressions to exclude struct packages and their names from checks.
# Regular expressions must match complete canonical struct package/name/structname.
# Default: []
exclude:
- '.+/cobra\.Command$'
forbidigo:
# Forbid the following identifiers (list of regexp).
# Default: ["^(fmt\\.Print(|f|ln)|print|println)$"]
forbid:
# Built-in bootstrapping functions.
- ^print(ln)?$
# Optional message that gets included in error reports.
- p: ^fmt\.Print.*$
msg: Do not commit print statements.
# Alternatively, put messages at the end of the regex, surrounded by `(# )?`
# Escape any special characters. Those messages get included in error reports.
- 'fmt\.Print.*(# Do not commit print statements\.)?'
# Forbid spew Dump, whether it is called as function or method.
# Depends on analyze-types below.
- ^spew\.(ConfigState\.)?Dump$
# The package name might be ambiguous.
# The full import path can be used as additional criteria.
# Depends on analyze-types below.
- p: ^v1.Dump$
pkg: ^example.com/pkg/api/v1$
# Exclude godoc examples from forbidigo checks.
# Default: true
exclude-godoc-examples: false
# Instead of matching the literal source code,
# use type information to replace expressions with strings that contain the package name
# and (for methods and fields) the type name.
# This makes it possible to handle import renaming and forbid struct fields and methods.
# Default: false
analyze-types: true
funlen:
# Checks the number of lines in a function.
# If lower than 0, disable the check.
# Default: 60
lines: -1
# Checks the number of statements in a function.
# If lower than 0, disable the check.
# Default: 40
statements: -1
# Ignore comments when counting lines.
# Default false
ignore-comments: true
gci:
# Section configuration to compare against.
# Section names are case-insensitive and may contain parameters in ().
# The default order of sections is `standard > default > custom > blank > dot > alias > localmodule`,
# If `custom-order` is `true`, it follows the order of `sections` option.
# Default: ["standard", "default"]
sections:
- standard # Standard section: captures all standard packages.
- default # Default section: contains all imports that could not be matched to another section type.
- prefix(github.com/org/project) # Custom section: groups all imports with the specified Prefix.
- blank # Blank section: contains all blank imports. This section is not present unless explicitly enabled.
- dot # Dot section: contains all dot imports. This section is not present unless explicitly enabled.
- alias # Alias section: contains all alias imports. This section is not present unless explicitly enabled.
- localmodule # Local module section: contains all local packages. This section is not present unless explicitly enabled.
# Checks that no inline Comments are present.
# Default: false
no-inline-comments: true
# Checks that no prefix Comments(comment lines above an import) are present.
# Default: false
no-prefix-comments: true
# Skip generated files.
# Default: true
skip-generated: false
# Enable custom order of sections.
# If `true`, make the section order the same as the order of `sections`.
# Default: false
custom-order: true
# Drops lexical ordering for custom sections.
# Default: false
no-lex-order: true
ginkgolinter:
# Suppress the wrong length assertion warning.
# Default: false
suppress-len-assertion: true
# Suppress the wrong nil assertion warning.
# Default: false
suppress-nil-assertion: true
# Suppress the wrong error assertion warning.
# Default: false
suppress-err-assertion: true
# Suppress the wrong comparison assertion warning.
# Default: false
suppress-compare-assertion: true
# Suppress the function all in async assertion warning.
# Default: false
suppress-async-assertion: true
# Suppress warning for comparing values from different types, like `int32` and `uint32`
# Default: false
suppress-type-compare-assertion: true
# Trigger warning for ginkgo focus containers like `FDescribe`, `FContext`, `FWhen` or `FIt`
# Default: false
forbid-focus-container: true
# Don't trigger warnings for HaveLen(0)
# Default: false
allow-havelen-zero: true
# Force using `Expect` with `To`, `ToNot` or `NotTo`.
# Reject using `Expect` with `Should` or `ShouldNot`.
# Default: false
force-expect-to: true
# Best effort validation of async intervals (timeout and polling).
# Ignored the suppress-async-assertion is true.
# Default: false
validate-async-intervals: true
# Trigger a warning for variable assignments in ginkgo containers like `Describe`, `Context` and `When`, instead of in `BeforeEach()`.
# Default: false
forbid-spec-pollution: true
# Force using the Succeed matcher for error functions, and the HaveOccurred matcher for non-function error values.
# Default: false
force-succeed: true
gochecksumtype:
# Presence of `default` case in switch statements satisfies exhaustiveness, if all members are not listed.
# Default: true
default-signifies-exhaustive: false
# Include shared interfaces in the exhaustiviness check.
# Default: false
include-shared-interfaces: true
gocognit:
# Minimal code complexity to report.
# Default: 30 (but we recommend 10-20)
min-complexity: 10
goconst:
# Minimal length of string constant.
# Default: 3
min-len: 2
# Minimum occurrences of constant string count to trigger issue.
# Default: 3
min-occurrences: 2
# Ignore test files.
# Default: false
ignore-tests: true
# Look for existing constants matching the values.
# Default: true
match-constant: false
# Search also for duplicated numbers.
# Default: false
numbers: true
# Minimum value, only works with goconst.numbers
# Default: 3
min: 2
# Maximum value, only works with goconst.numbers
# Default: 3
max: 2
# Ignore when constant is not used as function argument.
# Default: true
ignore-calls: false
# Exclude strings matching the given regular expression.
# Default: ""
ignore-strings: 'foo.+'
gocritic:
# Disable all checks.
# Default: false
disable-all: true
# Which checks should be enabled in addition to default checks; can't be combined with 'disabled-checks'.
# By default, list of stable checks is used (https://go-critic.com/overview#checks-overview):
# appendAssign, argOrder, assignOp, badCall, badCond, captLocal, caseOrder, codegenComment, commentFormatting,
# defaultCaseOrder, deprecatedComment, dupArg, dupBranchBody, dupCase, dupSubExpr, elseif, exitAfterDefer,
# flagDeref, flagName, ifElseChain, mapKey, newDeref, offBy1, regexpMust, singleCaseSwitch, sloppyLen,
# sloppyTypeAssert, switchTrue, typeSwitchVar, underef, unlambda, unslice, valSwap, wrapperFunc
# To see which checks are enabled run `GL_DEBUG=gocritic golangci-lint run --enable=gocritic`.
enabled-checks:
# Detects suspicious append result assignments.
# https://go-critic.com/overview.html#appendassign
- appendAssign
# Detects `append` chains to the same slice that can be done in a single `append` call.
# https://go-critic.com/overview.html#appendcombine
- appendCombine
# Detects suspicious arguments order.
# https://go-critic.com/overview.html#argorder
- argOrder
# Detects assignments that can be simplified by using assignment operators.
# https://go-critic.com/overview.html#assignop
- assignOp
# Detects suspicious function calls.
# https://go-critic.com/overview.html#badcall
- badCall
# Detects suspicious condition expressions.
# https://go-critic.com/overview.html#badcond
- badCond
# Detects suspicious mutex lock/unlock operations.
# https://go-critic.com/overview.html#badlock
- badLock
# Detects suspicious regexp patterns.
# https://go-critic.com/overview.html#badregexp
- badRegexp
# Detects bad usage of sort package.
# https://go-critic.com/overview.html#badsorting
- badSorting
# Detects bad usage of sync.OnceFunc.
# https://go-critic.com/overview.html#badsynconcefunc
- badSyncOnceFunc
# Detects bool expressions that can be simplified.
# https://go-critic.com/overview.html#boolexprsimplify
- boolExprSimplify
# Detects when predeclared identifiers are shadowed in assignments.
# https://go-critic.com/overview.html#builtinshadow
- builtinShadow
# Detects top-level declarations that shadow the predeclared identifiers.
# https://go-critic.com/overview.html#builtinshadowdecl
- builtinShadowDecl
# Detects capitalized names for local variables.
# https://go-critic.com/overview.html#captlocal
- captLocal
# Detects erroneous case order inside switch statements.
# https://go-critic.com/overview.html#caseorder
- caseOrder
# Detects malformed 'code generated' file comments.
# https://go-critic.com/overview.html#codegencomment
- codegenComment
# Detects comments with non-idiomatic formatting.
# https://go-critic.com/overview.html#commentformatting
- commentFormatting
# Detects commented-out code inside function bodies.
# https://go-critic.com/overview.html#commentedoutcode
- commentedOutCode
# Detects commented-out imports.
# https://go-critic.com/overview.html#commentedoutimport
- commentedOutImport
# Detects when default case in switch isn't on 1st or last position.
# https://go-critic.com/overview.html#defaultcaseorder
- defaultCaseOrder
# Detects loops inside functions that use defer.
# https://go-critic.com/overview.html#deferinloop
- deferInLoop
# Detects deferred function literals that can be simplified.
# https://go-critic.com/overview.html#deferunlambda
- deferUnlambda
# Detects malformed 'deprecated' doc-comments.
# https://go-critic.com/overview.html#deprecatedcomment
- deprecatedComment
# Detects comments that silence go lint complaints about doc-comment.
# https://go-critic.com/overview.html#docstub
- docStub
# Detects suspicious duplicated arguments.
# https://go-critic.com/overview.html#duparg
- dupArg
# Detects duplicated branch bodies inside conditional statements.
# https://go-critic.com/overview.html#dupbranchbody
- dupBranchBody
# Detects duplicated case clauses inside switch or select statements.
# https://go-critic.com/overview.html#dupcase
- dupCase
# Detects multiple imports of the same package under different aliases.
# https://go-critic.com/overview.html#dupimport
- dupImport
# Detects suspicious duplicated sub-expressions.
# https://go-critic.com/overview.html#dupsubexpr
- dupSubExpr
# Detects suspicious formatting strings usage.
# https://go-critic.com/overview.html#dynamicfmtstring
- dynamicFmtString
# Detects else with nested if statement that can be replaced with else-if.
# https://go-critic.com/overview.html#elseif
- elseif
# Detects suspicious empty declarations blocks.
# https://go-critic.com/overview.html#emptydecl
- emptyDecl
# Detects fallthrough that can be avoided by using multi case values.
# https://go-critic.com/overview.html#emptyfallthrough
- emptyFallthrough
# Detects empty string checks that can be written more idiomatically.
# https://go-critic.com/overview.html#emptystringtest
- emptyStringTest
# Detects unoptimal strings/bytes case-insensitive comparison.
# https://go-critic.com/overview.html#equalfold
- equalFold
# Detects unwanted dependencies on the evaluation order.
# https://go-critic.com/overview.html#evalorder
- evalOrder
# Detects calls to exit/fatal inside functions that use defer.
# https://go-critic.com/overview.html#exitafterdefer
- exitAfterDefer
# Detects exposed methods from sync.Mutex and sync.RWMutex.
# https://go-critic.com/overview.html#exposedsyncmutex
- exposedSyncMutex
# Detects suspicious reassignment of error from another package.
# https://go-critic.com/overview.html#externalerrorreassign
- externalErrorReassign
# Detects problems in filepath.Join() function calls.
# https://go-critic.com/overview.html#filepathjoin
- filepathJoin
# Detects immediate dereferencing of `flag` package pointers.
# https://go-critic.com/overview.html#flagderef
- flagDeref
# Detects suspicious flag names.
# https://go-critic.com/overview.html#flagname
- flagName
# Detects hex literals that have mixed case letter digits.
# https://go-critic.com/overview.html#hexliteral
- hexLiteral
# Detects nil usages in http.NewRequest calls, suggesting http.NoBody as an alternative.
# https://go-critic.com/overview.html#httpnobody
- httpNoBody
# Detects params that incur excessive amount of copying.
# https://go-critic.com/overview.html#hugeparam
- hugeParam
# Detects repeated if-else statements and suggests to replace them with switch statement.
# https://go-critic.com/overview.html#ifelsechain
- ifElseChain
# Detects when imported package names shadowed in the assignments.
# https://go-critic.com/overview.html#importshadow
- importShadow
# Detects strings.Index calls that may cause unwanted allocs.
# https://go-critic.com/overview.html#indexalloc
- indexAlloc
# Detects non-assignment statements inside if/switch init clause.
# https://go-critic.com/overview.html#initclause
- initClause
# Detects suspicious map literal keys.
# https://go-critic.com/overview.html#mapkey
- mapKey
# Detects method expression call that can be replaced with a method call.
# https://go-critic.com/overview.html#methodexprcall
- methodExprCall
# Finds where nesting level could be reduced.
# https://go-critic.com/overview.html#nestingreduce
- nestingReduce
# Detects immediate dereferencing of `new` expressions.
# https://go-critic.com/overview.html#newderef
- newDeref
# Detects return statements those results evaluate to nil.
# https://go-critic.com/overview.html#nilvalreturn
- nilValReturn
# Detects old-style octal literals.
# https://go-critic.com/overview.html#octalliteral
- octalLiteral
# Detects various off-by-one kind of errors.
# https://go-critic.com/overview.html#offby1
- offBy1
# Detects if function parameters could be combined by type and suggest the way to do it.
# https://go-critic.com/overview.html#paramtypecombine
- paramTypeCombine
# Detects expressions like []rune(s)[0] that may cause unwanted rune slice allocation.
# https://go-critic.com/overview.html#preferdecoderune
- preferDecodeRune
# Detects concatenation with os.PathSeparator which can be replaced with filepath.Join.
# https://go-critic.com/overview.html#preferfilepathjoin
- preferFilepathJoin
# Detects fmt.Sprint(f/ln) calls which can be replaced with fmt.Fprint(f/ln).
# https://go-critic.com/overview.html#preferfprint
- preferFprint
# Detects w.Write or io.WriteString calls which can be replaced with w.WriteString.
# https://go-critic.com/overview.html#preferstringwriter
- preferStringWriter
# Detects WriteRune calls with rune literal argument that is single byte and reports to use WriteByte instead.
# https://go-critic.com/overview.html#preferwritebyte
- preferWriteByte
# Detects input and output parameters that have a type of pointer to referential type.
# https://go-critic.com/overview.html#ptrtorefparam
- ptrToRefParam
# Detects append all its data while range it.
# https://go-critic.com/overview.html#rangeappendall
- rangeAppendAll
# Detects expensive copies of `for` loop range expressions.
# https://go-critic.com/overview.html#rangeexprcopy
- rangeExprCopy
# Detects loops that copy big objects during each iteration.
# https://go-critic.com/overview.html#rangevalcopy
- rangeValCopy
# Detects redundant fmt.Sprint calls.
# https://go-critic.com/overview.html#redundantsprint
- redundantSprint
# Detects `regexp.Compile*` that can be replaced with `regexp.MustCompile*`.
# https://go-critic.com/overview.html#regexpmust
- regexpMust
# Detects suspicious regexp patterns.
# https://go-critic.com/overview.html#regexppattern
- regexpPattern
# Detects regexp patterns that can be simplified.
# https://go-critic.com/overview.html#regexpsimplify
- regexpSimplify
# Detects suspicious http.Error call without following return.
# https://go-critic.com/overview.html#returnafterhttperror
- returnAfterHttpError
# Runs user-defined rules using ruleguard linter.
# https://go-critic.com/overview.html#ruleguard
- ruleguard
# Detects switch statements that could be better written as if statement.
# https://go-critic.com/overview.html#singlecaseswitch
- singleCaseSwitch
# Detects slice clear loops, suggests an idiom that is recognized by the Go compiler.
# https://go-critic.com/overview.html#sliceclear
- sliceClear
# Detects usage of `len` when result is obvious or doesn't make sense.
# https://go-critic.com/overview.html#sloppylen
- sloppyLen
# Detects suspicious/confusing re-assignments.
# https://go-critic.com/overview.html#sloppyreassign
- sloppyReassign
# Detects redundant type assertions.
# https://go-critic.com/overview.html#sloppytypeassert
- sloppyTypeAssert
# Detects suspicious sort.Slice calls.
# https://go-critic.com/overview.html#sortslice
- sortSlice
# Detects "%s" formatting directives that can be replaced with %q.
# https://go-critic.com/overview.html#sprintfquotedstring
- sprintfQuotedString
# Detects issue in Query() and Exec() calls.
# https://go-critic.com/overview.html#sqlquery
- sqlQuery
# Detects string concat operations that can be simplified.
# https://go-critic.com/overview.html#stringconcatsimplify
- stringConcatSimplify
# Detects redundant conversions between string and []byte.
# https://go-critic.com/overview.html#stringxbytes
- stringXbytes
# Detects strings.Compare usage.
# https://go-critic.com/overview.html#stringscompare
- stringsCompare
# Detects switch-over-bool statements that use explicit `true` tag value.
# https://go-critic.com/overview.html#switchtrue
- switchTrue
# Detects sync.Map load+delete operations that can be replaced with LoadAndDelete.
# https://go-critic.com/overview.html#syncmaploadanddelete
- syncMapLoadAndDelete
# Detects manual conversion to milli- or microseconds.
# https://go-critic.com/overview.html#timeexprsimplify
- timeExprSimplify
# Detects TODO comments without detail/assignee.
# https://go-critic.com/overview.html#todocommentwithoutdetail
- todoCommentWithoutDetail
# Detects function with too many results.
# https://go-critic.com/overview.html#toomanyresultschecker
- tooManyResultsChecker
# Detects potential truncation issues when comparing ints of different sizes.
# https://go-critic.com/overview.html#truncatecmp
- truncateCmp
# Detects repeated type assertions and suggests to replace them with type switch statement.
# https://go-critic.com/overview.html#typeassertchain
- typeAssertChain
# Detects method declarations preceding the type definition itself.
# https://go-critic.com/overview.html#typedeffirst
- typeDefFirst
# Detects type switches that can benefit from type guard clause with variable.
# https://go-critic.com/overview.html#typeswitchvar
- typeSwitchVar
# Detects unneeded parenthesis inside type expressions and suggests to remove them.
# https://go-critic.com/overview.html#typeunparen