review

Author: ldez

.golangci.yml

@@ -138,19 +138,30 @@ issues:
         - gomnd
 
     - path: pkg/golinters/errcheck.go
+      linters: [staticcheck]
       text: "SA1019: errCfg.Exclude is deprecated: use ExcludeFunctions instead"
     - path: pkg/commands/run.go
+      linters: [staticcheck]
       text: "SA1019: lsc.Errcheck.Exclude is deprecated: use ExcludeFunctions instead"
+    - path: pkg/commands/run.go
+      linters: [staticcheck]
+      text: "SA1019: c.cfg.Run.ShowStats is deprecated: use Output.ShowStats instead."
 
     - path: pkg/golinters/gofumpt.go
+      linters: [staticcheck]
       text: "SA1019: settings.LangVersion is deprecated: use the global `run.go` instead."
     - path: pkg/golinters/staticcheck_common.go
+      linters: [staticcheck]
       text: "SA1019: settings.GoVersion is deprecated: use the global `run.go` instead."
     - path: pkg/lint/lintersdb/manager.go
+      linters: [staticcheck]
       text: "SA1019: (.+).(GoVersion|LangVersion) is deprecated: use the global `run.go` instead."
+
     - path: pkg/golinters/unused.go
+      linters: [gocritic]
       text: "rangeValCopy: each iteration copies 160 bytes \\(consider pointers or indexing\\)"
     - path: test/(fix|linters)_test.go
+      linters: [gocritic]
       text: "string `gocritic.go` has 3 occurrences, make it a constant"
 
 run:

pkg/commands/run.go

@@ -398,7 +398,7 @@ func (c *runCommand) setExitCodeIfIssuesFound(issues []result.Issue) {
 
 func (c *runCommand) printStats(issues []result.Issue) {
 	if c.cfg.Run.ShowStats {
-		c.log.Warnf("run.show-stats is deprecated, please use output.show-stats")
+		c.log.Warnf("The configuration option `run.show-stats` is deprecated, please use `output.show-stats`")
 	}
 
 	if !c.cfg.Run.ShowStats && !c.cfg.Output.ShowStats {

pkg/config/run.go

@@ -23,7 +23,8 @@ type Run struct {
 	AllowParallelRunners bool `mapstructure:"allow-parallel-runners"`
 	AllowSerialRunners   bool `mapstructure:"allow-serial-runners"`
 
-	ShowStats bool `mapstructure:"show-stats"` // Deprecated use Output.ShowStats instead.
+	// Deprecated: use Output.ShowStats instead.
+	ShowStats bool `mapstructure:"show-stats"`
 
 	// It's obtain by flags and use for the tests and the context loader.
 	Args []string // Internal needs.