fix: ignore .WithStack and .WithMessagef from github.com/pkg/erro…

…rs (#15)
tomarrell · Jun 15, 2021 · 56ecd1e · 56ecd1e
1 parent 5a99e6a
commit 56ecd1e
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 1 deletion.
diff --git a/README.md b/README.md
@@ -35,6 +35,8 @@ ignoreSigs:
 - .Wrap(
 - .Wrapf(
 - .WithMessage(
+- .WithMessagef(
+- .WithStack(
 ```
 
 ## Usage

diff --git a/wrapcheck/testdata/ignore_pkg_errors/main.go b/wrapcheck/testdata/ignore_pkg_errors/main.go
@@ -24,5 +24,13 @@ func do() error {
 		return errors.WithMessage(err, "uh oh")
 	}
 
-	return nil
+	if err != nil {
+		return errors.WithMessagef(err, "uh %s", "oh")
+	}
+
+	if err != nil {
+		return errors.WithStack(err)
+	}
+
+	return errors.New("uh oh")
 }
diff --git a/wrapcheck/wrapcheck.go b/wrapcheck/wrapcheck.go
@@ -16,6 +16,8 @@ var DefaultIgnoreSigs = []string{
 	".Wrap(",
 	".Wrapf(",
 	".WithMessage(",
+	".WithMessagef(",
+	".WithStack(",
 }
 
 // WrapcheckConfig is the set of configuration values which configure the