Update Wrapcheck to v2, add configuration (#1947)

Author: tomarrell

.golangci.example.yml

@@ -550,6 +550,18 @@ linters-settings:
     multi-if: false   # Enforces newlines (or comments) after every multi-line if statement
     multi-func: false # Enforces newlines (or comments) after every multi-line function signature
 
+  wrapcheck:
+    # An array of strings that specify substrings of signatures to ignore.
+    # If this set, it will override the default set of ignored signatures.
+    # See https://github.com/tomarrell/wrapcheck#configuration for more information.
+    ignoreSigs:
+      - .Errorf(
+      - errors.New(
+      - errors.Unwrap(
+      - .Wrap(
+      - .Wrapf(
+      - .WithMessage(
+
   wsl:
     # See https://github.com/bombsimon/wsl/blob/master/doc/configuration.md for
     # documentation of available settings. These are the defaults for

go.mod

@@ -75,7 +75,7 @@ require (
 	github.com/tdakkota/asciicheck v0.0.0-20200416200610-e657995f937b
 	github.com/tetafro/godot v1.4.6
 	github.com/timakin/bodyclose v0.0.0-20200424151742-cb6215831a94
-	github.com/tomarrell/wrapcheck v1.2.0
+	github.com/tomarrell/wrapcheck/v2 v2.1.0
 	github.com/tommy-muehle/go-mnd/v2 v2.3.2
 	github.com/ultraware/funlen v0.0.3
 	github.com/ultraware/whitespace v0.0.4

go.sum

@@ -130,6 +130,7 @@ type LintersSettings struct {
 	Unused           StaticCheckSettings
 	Varcheck         VarCheckSettings
 	Whitespace       WhitespaceSettings
+	Wrapcheck        WrapcheckSettings
 	WSL              WSLSettings
 
 	Custom map[string]CustomLinterSettings
@@ -430,6 +431,10 @@ type WhitespaceSettings struct {
 	MultiFunc bool `mapstructure:"multi-func"`
 }
 
+type WrapcheckSettings struct {
+	IgnoreSigs []string `mapstructure:"ignoreSigs"`
+}
+
 type WSLSettings struct {
 	StrictAppend                     bool `mapstructure:"strict-append"`
 	AllowAssignAndCallCuddle         bool `mapstructure:"allow-assign-and-call"`

pkg/config/linters_settings.go

@@ -1,19 +1,29 @@
 package golinters
 
 import (
-	"github.com/tomarrell/wrapcheck/wrapcheck"
+	"github.com/tomarrell/wrapcheck/v2/wrapcheck"
 	"golang.org/x/tools/go/analysis"
 
+	"github.com/golangci/golangci-lint/pkg/config"
 	"github.com/golangci/golangci-lint/pkg/golinters/goanalysis"
 )
 
 const wrapcheckName = "wrapcheck"
 
-func NewWrapcheck() *goanalysis.Linter {
+func NewWrapcheck(settings *config.WrapcheckSettings) *goanalysis.Linter {
+	cfg := wrapcheck.NewDefaultConfig()
+	if settings != nil {
+		if len(settings.IgnoreSigs) != 0 {
+			cfg.IgnoreSigs = settings.IgnoreSigs
+		}
+	}
+
+	a := wrapcheck.NewAnalyzer(cfg)
+
 	return goanalysis.NewLinter(
 		wrapcheckName,
-		wrapcheck.Analyzer.Doc,
-		[]*analysis.Analyzer{wrapcheck.Analyzer},
+		a.Doc,
+		[]*analysis.Analyzer{a},
 		nil,
 	).WithLoadMode(goanalysis.LoadModeTypesInfo)
 }

pkg/golinters/wrapcheck.go

@@ -117,6 +117,7 @@ func (m Manager) GetAllSupportedLinterConfigs() []*linter.Config {
 	var staticcheckCfg *config.StaticCheckSettings
 	var stylecheckCfg *config.StaticCheckSettings
 	var unusedCfg *config.StaticCheckSettings
+	var wrapcheckCfg *config.WrapcheckSettings
 
 	if m.cfg != nil {
 		govetCfg = &m.cfg.LintersSettings.Govet
@@ -137,6 +138,7 @@ func (m Manager) GetAllSupportedLinterConfigs() []*linter.Config {
 		staticcheckCfg = &m.cfg.LintersSettings.Staticcheck
 		stylecheckCfg = &m.cfg.LintersSettings.Stylecheck
 		unusedCfg = &m.cfg.LintersSettings.Unused
+		wrapcheckCfg = &m.cfg.LintersSettings.Wrapcheck
 	}
 
 	const megacheckName = "megacheck"
@@ -411,7 +413,7 @@ func (m Manager) GetAllSupportedLinterConfigs() []*linter.Config {
 			WithSince("v1.30.0").
 			WithPresets(linter.PresetStyle).
 			WithURL("https://github.com/ssgreg/nlreturn"),
-		linter.NewConfig(golinters.NewWrapcheck()).
+		linter.NewConfig(golinters.NewWrapcheck(wrapcheckCfg)).
 			WithSince("v1.32.0").
 			WithPresets(linter.PresetStyle, linter.PresetError).
 			WithLoadForGoAnalysis().