review: update implementation

Author: ldez

.golangci.reference.yml

@@ -1159,6 +1159,10 @@ linters-settings:
     require-specific: true
 
   nonamedreturns:
+    # Report named error if it is assigned inside defer.
+    # Default: false
+    report-error-in-defer: true
+    # DEPRECATED use report-error-in-defer instead.
     # Do not complain about named error, if it is assigned inside defer.
     # Default: false
     allow-error-in-defer: true

pkg/config/linters_settings.go

@@ -486,7 +486,8 @@ type NoLintLintSettings struct {
 }
 
 type NoNamedReturnsSettings struct {
-	AllowErrorInDefer bool `mapstructure:"allow-error-in-defer"`
+	AllowErrorInDefer  bool `mapstructure:"allow-error-in-defer"` // Deprecated: use ReportErrorInDefer instead.
+	ReportErrorInDefer bool `mapstructure:"report-error-in-defer"`
 }
 type ParallelTestSettings struct {
 	IgnoreMissing bool `mapstructure:"ignore-missing"`

pkg/golinters/nonamedreturns.go

@@ -15,7 +15,7 @@ func NewNoNamedReturns(settings *config.NoNamedReturnsSettings) *goanalysis.Lint
 	if settings != nil {
 		cfg = map[string]map[string]interface{}{
 			a.Name: {
-				analyzer.FlagAllowErrorInDefer: settings.AllowErrorInDefer,
+				analyzer.FlagReportErrorInDefer: settings.ReportErrorInDefer || settings.AllowErrorInDefer,
 			},
 		}
 	}

test/testdata/configs/nonamedreturns.yml

@@ -1,3 +1,3 @@
 linters-settings:
   nonamedreturns:
-    allow-error-in-defer: true
+    report-error-in-defer: true