add nlreturn

Author: remyleone

.golangci.yml

@@ -49,6 +49,7 @@ linters:
     - musttag # enforce field tags in (un)marshaled structs [fast: false, auto-fix: false]
     - nakedret # Finds naked returns in functions greater than a specified function length [fast: true, auto-fix: false]
     - nilerr # Finds the code that returns nil even if it checks that the error is not nil. [fast: false, auto-fix: false]
+    - nlreturn # nlreturn checks for a new line before return and branch statements to increase code clarity [fast: true, auto-fix: true]
     - noctx # noctx finds sending http request without context.Context [fast: false, auto-fix: false]
     - nolintlint # Reports ill-formed or insufficient nolint directives [fast: true, auto-fix: false]
     - nosprintfhostport # Checks for misuse of Sprintf to construct a host with port in a URL. [fast: true, auto-fix: false]
@@ -99,7 +100,6 @@ linters:
     - mnd # An analyzer to detect magic numbers. [fast: true, auto-fix: false]
     - nestif # Reports deeply nested if statements [fast: true, auto-fix: false]
     - nilnil # Checks that there is no simultaneous return of `nil` error and an invalid value. [fast: false, auto-fix: false]
-    - nlreturn # nlreturn checks for a new line before return and branch statements to increase code clarity [fast: true, auto-fix: false]
     - varnamelen # checks that the length of a variable's name matches its scope [fast: false, auto-fix: false]
     - wsl # Whitespace Linter - Forces you to use empty lines! [fast: true, auto-fix: false]
 

internal/acctest/acctest.go

@@ -98,6 +98,7 @@ func extractGeneratedNamePrefix(name string) string {
 	//         ^
 	dashIndex = strings.LastIndex(name, "-")
 	name = name[:dashIndex]
+
 	return name
 }
 
@@ -155,6 +156,7 @@ func compareJSONBodies(expected, actual map[string]interface{}) bool {
 			return false
 		}
 	}
+
 	return true
 }
 

internal/acctest/checks.go

@@ -26,6 +26,7 @@ func CheckResourceIDChanged(resourceName string, resourceID *string) resource.Te
 			return errors.New("resource ID persisted when it should have changed")
 		}
 		*resourceID = rs.Primary.ID
+
 		return nil
 	}
 }
@@ -42,6 +43,7 @@ func CheckResourceIDPersisted(resourceName string, resourceID *string) resource.
 			return errors.New("resource ID changed when it should have persisted")
 		}
 		*resourceID = rs.Primary.ID
+
 		return nil
 	}
 }
@@ -90,6 +92,7 @@ func CheckResourceAttrFunc(name string, key string, test func(string) error) res
 		if err != nil {
 			return fmt.Errorf("test for %s %s did not pass test: %s", name, key, err)
 		}
+
 		return nil
 	}
 }
@@ -100,6 +103,7 @@ func CheckResourceAttrIPv4(name string, key string) resource.TestCheckFunc {
 		if ip.To4() == nil {
 			return fmt.Errorf("%s is not a valid IPv4", value)
 		}
+
 		return nil
 	})
 }
@@ -110,6 +114,7 @@ func CheckResourceAttrIPv6(name string, key string) resource.TestCheckFunc {
 		if ip.To16() == nil {
 			return fmt.Errorf("%s is not a valid IPv6", value)
 		}
+
 		return nil
 	})
 }
@@ -120,6 +125,7 @@ func CheckResourceAttrIP(name string, key string) resource.TestCheckFunc {
 		if ip == nil {
 			return fmt.Errorf("%s is not a valid IP", value)
 		}
+
 		return nil
 	})
 }

internal/acctest/domain.go

@@ -36,12 +36,14 @@ func init() {
 	for _, reservedDomain := range reservedDomains {
 		if reservedDomain.MatchString(TestDomain) {
 			isReserved = true
+
 			break
 		}
 	}
 
 	if isReserved {
 		logging.L.Warningf("TF_TEST_DOMAIN cannot be a Scaleway required domain. Please use another one.")
+
 		return
 	}
 

internal/acctest/sweepers.go

@@ -16,6 +16,7 @@ func Sweep(f func(scwClient *scw.Client) error) error {
 	if err != nil {
 		return err
 	}
+
 	return f(m.ScwClient())
 }
 
@@ -30,6 +31,7 @@ func SweepZones(zones []scw.Zone, f func(scwClient *scw.Client, zone scw.Zone) e
 			logging.L.Warningf("error running sweepZones, ignoring: %s", err)
 		}
 	}
+
 	return nil
 }
 
@@ -41,6 +43,7 @@ func SweepRegions(regions []scw.Region, f func(scwClient *scw.Client, region scw
 
 	return SweepZones(zones, func(scwClient *scw.Client, zone scw.Zone) error {
 		r, _ := zone.Region()
+
 		return f(scwClient, r)
 	})
 }
@@ -56,5 +59,6 @@ func sharedClientForZone(zone scw.Zone) (*scw.Client, error) {
 	if err != nil {
 		return nil, err
 	}
+
 	return m.ScwClient(), nil
 }

internal/acctest/validate_cassettes_test.go

@@ -40,6 +40,7 @@ func getTestFiles() (map[string]struct{}, error) {
 		if isCassette && !isException {
 			filesMap[fileNameWithoutExtSuffix(path)] = struct{}{}
 		}
+
 		return nil
 	})
 	if err != nil {
@@ -87,8 +88,10 @@ func checkErrCodeExcept(i *cassette.Interaction, c *cassette.Cassette, codes ...
 				return true
 			}
 		}
+
 		return false
 	}
+
 	return true
 }
 

internal/acctest/vcr.go

@@ -70,6 +70,7 @@ func compareJSONFields(expected, actualI interface{}) bool {
 		if _, isString := expected.(string); !isString {
 			return false
 		}
+
 		return compareJSONFieldsStrings(expected.(string), actual)
 	default:
 		// Consider equality when not handled
@@ -116,6 +117,7 @@ func cassetteBodyMatcher(actualRequest *http.Request, cassetteRequest cassette.R
 		} else if _, isFile := actualRequest.Body.(*os.File); isFile {
 			return true // Body match if request is sending a file, maybe do more check here
 		}
+
 		return false
 	}
 
@@ -243,6 +245,7 @@ func cassetteSensitiveFieldsAnonymizer(i *cassette.Interaction) error {
 		return fmt.Errorf("failed to marshal anonymized body: %w", err)
 	}
 	i.Response.Body = string(anonymizedBody)
+
 	return nil
 }
 
@@ -290,6 +293,7 @@ func getHTTPRecoder(t *testing.T, pkgFolder string, update bool) (client *http.C
 		delete(i.Request.Headers, "x-auth-token")
 		delete(i.Request.Headers, "X-Auth-Token")
 		delete(i.Request.Headers, "Authorization")
+
 		return nil
 	}, recorder.BeforeSaveHook)
 

internal/cdf/locality.go

@@ -57,6 +57,7 @@ func getLocality(diff *schema.ResourceDiff, m interface{}) string {
 		region, _ := meta.ExtractRegion(diff, m)
 		loc = region.String()
 	}
+
 	return loc
 }
 
@@ -91,6 +92,7 @@ func LocalityCheck(keys ...string) schema.CustomizeDiffFunc {
 				}
 			}
 		}
+
 		return nil
 	}
 }

internal/datasource/schemas.go

@@ -49,6 +49,7 @@ func SchemaFromResourceSchema(rs map[string]*schema.Schema) map[string]*schema.S
 		switch v.Type {
 		case schema.TypeSet:
 			dv.Set = v.Set
+
 			fallthrough
 		case schema.TypeList:
 			// List & Set types are generally used for 2 cases:
@@ -70,6 +71,7 @@ func SchemaFromResourceSchema(rs map[string]*schema.Schema) map[string]*schema.S
 		}
 		ds[k] = dv
 	}
+
 	return ds
 }
 

internal/datasource/search.go

@@ -18,6 +18,7 @@ func FindExact[T any](slice []T, finder func(T) bool, searchName string) (T, err
 			if foundFlag {
 				// More than one element found with the same search name
 				var zero T
+
 				return zero, fmt.Errorf("multiple elements found with the name %s", searchName)
 			}
 			found = elem
@@ -27,6 +28,7 @@ func FindExact[T any](slice []T, finder func(T) bool, searchName string) (T, err
 
 	if !foundFlag {
 		var zero T
+
 		return zero, fmt.Errorf("no element found with the name %s", searchName)
 	}
 
@@ -51,6 +53,7 @@ func SingularDataSourceFindError(resourceType string, err error) error {
 // retry.NotFoundError.
 func notFound(err error) bool {
 	var e *retry.NotFoundError // nosemgrep:ci.is-not-found-error
+
 	return errors.As(err, &e)
 }
 
@@ -65,6 +68,7 @@ func (e *TooManyResultsError) Error() string {
 
 func (e *TooManyResultsError) Is(err error) bool {
 	_, ok := err.(*TooManyResultsError) //nolint:errorlint // Explicitly does *not* match down the error tree
+
 	return ok
 }