chore: bump golangci-lint to 2.5.0 (#3358)

Author: remyleone

.github/workflows/golangci-lint.yml

@@ -15,5 +15,5 @@ jobs:
       - name: golangci-lint
         uses: golangci/[email protected]
         with:
-          version: v2.3.0
+          version: v2.5.0
           args: --timeout 5m

internal/locality/regional/ids.go

@@ -62,22 +62,22 @@ func NewIDString(region scw.Region, id string) string {
 func ParseNestedID(regionalNestedID string) (region scw.Region, outerID, innerID string, err error) {
 	loc, innerID, outerID, err := locality.ParseLocalizedNestedID(regionalNestedID)
 	if err != nil {
-		return
+		return region, outerID, innerID, err
 	}
 
 	region, err = scw.ParseRegion(loc)
 
-	return
+	return region, outerID, innerID, err
 }
 
 // ParseID parses a regionalID and extracts the resource region and id.
 func ParseID(regionalID string) (region scw.Region, id string, err error) {
 	loc, id, err := locality.ParseLocalizedID(regionalID)
 	if err != nil {
-		return
+		return region, id, err
 	}
 
 	region, err = scw.ParseRegion(loc)
 
-	return
+	return region, id, err
 }

internal/locality/zonal/ids.go

@@ -73,17 +73,17 @@ func ParseID(zonedID string) (zone scw.Zone, id string, err error) {
 
 	zone, err = scw.ParseZone(rawZone)
 
-	return
+	return zone, id, err
 }
 
 // ParseNestedID parses a zonedNestedID and extracts the resource zone ,inner and outer ID.
 func ParseNestedID(zonedNestedID string) (zone scw.Zone, outerID, innerID string, err error) {
 	rawZone, innerID, outerID, err := locality.ParseLocalizedNestedID(zonedNestedID)
 	if err != nil {
-		return
+		return zone, outerID, innerID, err
 	}
 
 	zone, err = scw.ParseZone(rawZone)
 
-	return
+	return zone, outerID, innerID, err
 }

internal/services/iam/group_membership.go

@@ -178,7 +178,7 @@ func ExpandGroupMembershipID(id string) (groupID string, userID string, applicat
 		applicationID = elems[2]
 	}
 
-	return
+	return groupID, userID, applicationID, err
 }
 
 func MakeGroupRequest(ctx context.Context, api *iam.API, request any) (*iam.Group, error) {

internal/services/instance/server_test.go

@@ -2023,6 +2023,7 @@ func TestAccServer_PrivateNetworkMissingPNIC(t *testing.T) {
 				ImportState:  true,
 				ImportStateIdFunc: func(state *terraform.State) (string, error) {
 					serverID := state.RootModule().Resources["scaleway_instance_server.main"].Primary.ID
+
 					pnicID, exists := state.RootModule().Resources["scaleway_instance_server.main"].Primary.Attributes["private_network.0.pnic_id"]
 					if !exists {
 						return "", errors.New("private_network.0.pnic_id not found")
@@ -2066,7 +2067,9 @@ func TestAccServer_PrivateNetworkMissingPNIC(t *testing.T) {
 						if !exists {
 							return errors.New("private_network.0.pnic_id not found")
 						}
+
 						localizedPNICID := state.RootModule().Resources["scaleway_instance_private_nic.pnic"].Primary.ID
+
 						_, pnicID, _, err := zonal.ParseNestedID(localizedPNICID)
 						if err != nil {
 							return err

internal/services/k8s/cluster.go

@@ -233,6 +233,7 @@ func ResourceCluster() *schema.Resource {
 				if okAutoUpgradeEnable && autoUpgradeEnable.(bool) && !versionIsOnlyMinor {
 					return errors.New("only minor version x.y can be used with auto upgrade enabled")
 				}
+
 				if versionIsOnlyMinor && !autoUpgradeEnable.(bool) {
 					return errors.New("minor version x.y must only be used with auto upgrade enabled")
 				}
@@ -257,11 +258,13 @@ func ResourceCluster() *schema.Resource {
 							// If no private network has been set yet, migrate the cluster in the Update function
 							return nil
 						}
+
 						if planned != "" {
 							_, plannedPNID, err := locality.ParseLocalizedID(planned.(string))
 							if err != nil {
 								return err
 							}
+
 							if plannedPNID == actual {
 								// If the private network ID is the same, do nothing
 								return nil
@@ -283,6 +286,7 @@ func ResourceCluster() *schema.Resource {
 					if err != nil {
 						return err
 					}
+
 					possibleTypes, err := k8sAPI.ListClusterAvailableTypes(&k8s.ListClusterAvailableTypesRequest{
 						Region:    region,
 						ClusterID: clusterID,
@@ -297,6 +301,7 @@ func ResourceCluster() *schema.Resource {
 							return nil
 						}
 					}
+
 					err = diff.ForceNew("type")
 					if err != nil {
 						return err

internal/services/lb/backends_data_source.go

@@ -62,6 +62,7 @@ func DataSourceBackends() *schema.Resource {
 							Computed: true,
 							Description: func() string {
 								var t lb.ForwardPortAlgorithm
+
 								values := t.Values()
 
 								return fmt.Sprintf("Load balancing algorithm to be used when determining which backend server to forward new traffic to. Possible values are: %s", values)

internal/services/lb/lb_test.go

@@ -281,6 +281,7 @@ func TestAccLB_Migrate(t *testing.T) {
 						if !ok {
 							return fmt.Errorf("resource not found: %s", "scaleway_lb.main")
 						}
+
 						lbID = rs.Primary.ID
 
 						return nil
@@ -310,6 +311,7 @@ func TestAccLB_Migrate(t *testing.T) {
 						if !ok {
 							return fmt.Errorf("resource not found: %s", "scaleway_lb.main")
 						}
+
 						if rs.Primary.ID != lbID {
 							return errors.New("LB id has changed")
 						}

internal/services/lb/lbs_data_source.go

@@ -165,6 +165,7 @@ func DataSourceLbs() *schema.Resource {
 							Computed: true,
 							Description: func() string {
 								var t lb.SSLCompatibilityLevel
+
 								values := t.Values()
 
 								return fmt.Sprintf("SSL compatibility level possible values are %s", values)

internal/services/mnq/sns_topic_test.go

@@ -124,6 +124,7 @@ func TestAccSNSTopic_Basic(t *testing.T) {
 						if !exists {
 							return errors.New("failed to find resource")
 						}
+
 						name, exists := topic.Primary.Attributes["name"]
 						if !exists {
 							return errors.New("failed to find atttribute")