update: nlreturn

adds configuration `block-size` that defines size of the block
(including return statement that are still "OK") so no return
split required.

Author: butuzov

.golangci.example.yml

@@ -158,6 +158,11 @@ linters-settings:
     # Exclude godoc examples from forbidigo checks.  Default is true.
     exclude_godoc_examples: false
 
+  nlreturn:
+    # size of the block (including return statement that are still "OK")
+    # so no return split required.
+    block-size: 1
+
   funlen:
     lines: 60
     statements: 40

pkg/config/linters_settings.go

@@ -118,6 +118,7 @@ type LintersSettings struct {
 	Misspell         MisspellSettings
 	Nakedret         NakedretSettings
 	Nestif           NestifSettings
+	Nlreturn         NlreturnSettings
 	NoLintLint       NoLintLintSettings
 	Prealloc         PreallocSettings
 	Predeclared      PredeclaredSettings
@@ -360,6 +361,10 @@ type NestifSettings struct {
 	MinComplexity int `mapstructure:"min-complexity"`
 }
 
+type NlreturnSettings struct {
+	BlockSize int `mapstructure:"block-size"`
+}
+
 type NoLintLintSettings struct {
 	RequireExplanation bool     `mapstructure:"require-explanation"`
 	AllowLeadingSpace  bool     `mapstructure:"allow-leading-space"`

pkg/golinters/nlreturn.go

@@ -4,16 +4,24 @@ import (
 	"github.com/ssgreg/nlreturn/v2/pkg/nlreturn"
 	"golang.org/x/tools/go/analysis"
 
+	"github.com/golangci/golangci-lint/pkg/config"
 	"github.com/golangci/golangci-lint/pkg/golinters/goanalysis"
 )
 
-func NewNLReturn() *goanalysis.Linter {
+func NewNLReturn(settings *config.NlreturnSettings) *goanalysis.Linter {
+	a := nlreturn.NewAnalyzer()
+
+	cfg := map[string]map[string]interface{}{}
+	if settings != nil {
+		cfg[a.Name] = map[string]interface{}{
+			"block-size": settings.BlockSize,
+		}
+	}
+
 	return goanalysis.NewLinter(
-		"nlreturn",
+		a.Name,
 		"nlreturn checks for a new line before return and branch statements to increase code clarity",
-		[]*analysis.Analyzer{
-			nlreturn.NewAnalyzer(),
-		},
-		nil,
+		[]*analysis.Analyzer{a},
+		cfg,
 	).WithLoadMode(goanalysis.LoadModeSyntax)
 }

pkg/lint/lintersdb/manager.go

@@ -119,6 +119,7 @@ func (m Manager) GetAllSupportedLinterConfigs() []*linter.Config {
 	var stylecheckCfg *config.StaticCheckSettings
 	var unusedCfg *config.StaticCheckSettings
 	var wrapcheckCfg *config.WrapcheckSettings
+	var nlreturnCfg *config.NlreturnSettings
 
 	if m.cfg != nil {
 		govetCfg = &m.cfg.LintersSettings.Govet
@@ -141,6 +142,7 @@ func (m Manager) GetAllSupportedLinterConfigs() []*linter.Config {
 		stylecheckCfg = &m.cfg.LintersSettings.Stylecheck
 		unusedCfg = &m.cfg.LintersSettings.Unused
 		wrapcheckCfg = &m.cfg.LintersSettings.Wrapcheck
+		nlreturnCfg = &m.cfg.LintersSettings.Nlreturn
 	}
 
 	const megacheckName = "megacheck"
@@ -412,7 +414,7 @@ func (m Manager) GetAllSupportedLinterConfigs() []*linter.Config {
 			WithPresets(linter.PresetBugs, linter.PresetSQL).
 			WithLoadForGoAnalysis().
 			WithURL("https://github.com/ryanrolds/sqlclosecheck"),
-		linter.NewConfig(golinters.NewNLReturn()).
+		linter.NewConfig(golinters.NewNLReturn(nlreturnCfg)).
 			WithSince("v1.30.0").
 			WithPresets(linter.PresetStyle).
 			WithURL("https://github.com/ssgreg/nlreturn"),

test/testdata/configs/nlreturn.yml

@@ -0,0 +1,3 @@
+linters-settings:
+  nlreturn:
+    block-size: 2

test/testdata/nlreturn-block-size.go

@@ -0,0 +1,22 @@
+// args: -Enlreturn
+// config_path: testdata/configs/nlreturn.yml
+package testdata
+
+func foo0(n int) int {
+	if n == 1 {
+		n2 := n * n
+		return n2
+	}
+
+	return 1
+}
+
+func foo1(n int) int {
+	if n == 1 {
+		n2 := n * n
+		n3 := n2 * n
+		return n3 // ERROR "return with no blank line before"
+	}
+
+	return 1
+}