add noinlineerr linter

Author: AlwxSin

.golangci.next.reference.yml

@@ -91,6 +91,7 @@ linters:
     - nilnil
     - nlreturn
     - noctx
+    - noinlineerr
     - nolintlint
     - nonamedreturns
     - nosprintfhostport
@@ -200,6 +201,7 @@ linters:
     - nilnil
     - nlreturn
     - noctx
+    - noinlineerr
     - nolintlint
     - nonamedreturns
     - nosprintfhostport

go.mod

@@ -7,6 +7,7 @@ require (
 	4d63.com/gochecknoglobals v0.2.2
 	github.com/4meepo/tagalign v1.4.2
 	github.com/Abirdcfly/dupword v0.1.4
+	github.com/AlwxSin/noinlineerr v1.0.1
 	github.com/Antonboom/errname v1.1.0
 	github.com/Antonboom/nilnil v1.1.0
 	github.com/Antonboom/testifylint v1.6.1

go.sum

@@ -791,6 +791,7 @@
             "nilnil",
             "nlreturn",
             "noctx",
+            "noinlineerr",
             "nolintlint",
             "nonamedreturns",
             "nosprintfhostport",

jsonschema/golangci.next.jsonschema.json

@@ -0,0 +1,18 @@
+package noinlineerr
+
+import (
+	"github.com/AlwxSin/noinlineerr"
+	"golang.org/x/tools/go/analysis"
+
+	"github.com/golangci/golangci-lint/v2/pkg/goanalysis"
+)
+
+func New() *goanalysis.Linter {
+	analyzer := noinlineerr.NewAnalyzer()
+	return goanalysis.NewLinter(
+		analyzer.Name,
+		analyzer.Doc,
+		[]*analysis.Analyzer{analyzer},
+		nil,
+	).WithLoadMode(goanalysis.LoadModeTypesInfo)
+}

pkg/golinters/noinlineerr/noinlineerr.go

@@ -0,0 +1,11 @@
+package noinlineerr
+
+import (
+	"testing"
+
+	"github.com/golangci/golangci-lint/v2/test/testshared/integration"
+)
+
+func TestFromTestdata(t *testing.T) {
+	integration.RunTestdata(t)
+}

pkg/golinters/noinlineerr/noinlineerr_integration_test.go

@@ -0,0 +1,52 @@
+//golangcitest:args -Enoinlineerr
+package testdata
+
+func doSomething() error {
+	return nil
+}
+
+func doSmthManyArgs(a, b, c, d int) error {
+	return nil
+}
+
+func doSmthMultipleReturn() (bool, error) {
+	return false, nil
+}
+
+func valid() error {
+	err := doSomething() // ok
+	if err != nil {
+		return err
+	}
+
+	err = doSmthManyArgs(0, 0, 0, 0) // ok
+	if err != nil {
+		return err
+	}
+
+	_, err = doSmthMultipleReturn() // ok
+	if err != nil {
+		return err
+	}
+	return nil
+}
+
+func invalid() error {
+	if err := doSomething(); err != nil { // want "avoid inline error handling using `if err := ...; err != nil; use plain assignment `err := ..."
+		return err
+	}
+
+	if err := doSmthManyArgs(0, // want "avoid inline error handling using `if err := ...; err != nil; use plain assignment `err := ..."
+		0,
+		0,
+		0,
+	); err != nil {
+		return err
+	}
+
+	if _, err := doSmthMultipleReturn(); err != nil { // want "avoid inline error handling using `if err := ...; err != nil; use plain assignment `err := ..."
+		_ = false
+		return err
+	}
+	return nil
+}

pkg/golinters/noinlineerr/testdata/noinlineerr.go

@@ -79,6 +79,7 @@ import (
 	"github.com/golangci/golangci-lint/v2/pkg/golinters/nilnil"
 	"github.com/golangci/golangci-lint/v2/pkg/golinters/nlreturn"
 	"github.com/golangci/golangci-lint/v2/pkg/golinters/noctx"
+	"github.com/golangci/golangci-lint/v2/pkg/golinters/noinlineerr"
 	"github.com/golangci/golangci-lint/v2/pkg/golinters/nolintlint"
 	"github.com/golangci/golangci-lint/v2/pkg/golinters/nonamedreturns"
 	"github.com/golangci/golangci-lint/v2/pkg/golinters/nosprintfhostport"
@@ -511,6 +512,11 @@ func (LinterBuilder) Build(cfg *config.Config) ([]*linter.Config, error) {
 			WithLoadForGoAnalysis().
 			WithURL("https://github.com/sonatard/noctx"),
 
+		linter.NewConfig(noinlineerr.New()).
+			WithSince("v2.2.80").
+			WithLoadForGoAnalysis().
+			WithURL("https://github.com/AlwxSin/noinlineerr"),
+
 		linter.NewConfig(nonamedreturns.New(&cfg.Linters.Settings.NoNamedReturns)).
 			WithSince("v1.46.0").
 			WithLoadForGoAnalysis().