Update dependency golangci/golangci-lint to v2.7.2

.github/workflows/lint.yml

@@ -50,7 +50,7 @@ jobs:
         uses: golangci/golangci-lint-action@1e7e51e771db61008b38414a730f564565cf7c20 # v9.2.0
         with:
           working-directory: ${{ matrix.directory }}
-          version: v2.6.2 # renovate: datasource=github-tags depName=golangci/golangci-lint
+          version: v2.7.2 # renovate: datasource=github-tags depName=golangci/golangci-lint
 
   njs-lint:
     name: NJS Lint

.pre-commit-config.yaml

@@ -38,7 +38,7 @@ repos:
           - javascript
 
   - repo: https://github.com/golangci/golangci-lint
-    rev: v2.6.2
+    rev: v2.7.2
     hooks:
       - id: golangci-lint-full
         name: golangci-lint-root

Makefile

@@ -24,7 +24,7 @@ GO_LINKER_FLAGS = $(GO_LINKER_FLAGS_OPTIMIZATIONS) $(GO_LINKER_FlAGS_VARS)
 
 # tools versions
 # renovate: datasource=github-tags depName=golangci/golangci-lint
-GOLANGCI_LINT_VERSION = v2.6.2
+GOLANGCI_LINT_VERSION = v2.7.2
 # renovate: datasource=docker depName=kindest/node
 KIND_K8S_VERSION = v1.34.0
 # renovate: datasource=github-tags depName=norwoodj/helm-docs

internal/controller/ngfsort/doc.go

@@ -0,0 +1,2 @@
+// Package ngfsort provides utilities for sorting Kubernetes resources.
+package ngfsort

internal/controller/sort/sort.go → internal/controller/ngfsort/sort.go

@@ -1,4 +1,4 @@
-package sort
+package ngfsort
 
 import (
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"

internal/controller/sort/sort_test.go → internal/controller/ngfsort/sort_test.go

@@ -1,4 +1,4 @@
-package sort
+package ngfsort
 
 import (
 	"testing"

internal/controller/nginx/agent/grpc/context/doc.go

@@ -1,4 +1,4 @@
 /*
 Package context contains the functions for storing extra information in the gRPC context.
 */
-package context
+package context //nolint:revive // ignoring conflicting package name

internal/controller/nginx/config/http/config.go

@@ -1,4 +1,4 @@
-package http
+package http //nolint:revive // ignoring conflicting package name
 
 import (
 	"github.com/nginx/nginx-gateway-fabric/v2/internal/controller/nginx/config/shared"

internal/controller/nginx/config/shared/config.go

@@ -1,4 +1,4 @@
-package shared
+package shared //nolint:revive // ignoring meaningless package name
 
 // Map defines an NGINX map.
 type Map struct {

internal/controller/state/dataplane/sort.go

@@ -3,7 +3,7 @@ package dataplane
 import (
 	"sort"
 
-	ngfsort "github.com/nginx/nginx-gateway-fabric/v2/internal/controller/sort"
+	"github.com/nginx/nginx-gateway-fabric/v2/internal/controller/ngfsort"
 )
 
 func sortMatchRules(matchRules []MatchRule) {

internal/controller/state/graph/backend_refs.go

@@ -12,7 +12,7 @@ import (
 	gatewayv1 "sigs.k8s.io/gateway-api/apis/v1"
 
 	ngfAPIv1alpha2 "github.com/nginx/nginx-gateway-fabric/v2/apis/v1alpha2"
-	"github.com/nginx/nginx-gateway-fabric/v2/internal/controller/sort"
+	sort "github.com/nginx/nginx-gateway-fabric/v2/internal/controller/ngfsort"
 	"github.com/nginx/nginx-gateway-fabric/v2/internal/controller/state/conditions"
 	"github.com/nginx/nginx-gateway-fabric/v2/internal/framework/helpers"
 	"github.com/nginx/nginx-gateway-fabric/v2/internal/framework/kinds"

internal/controller/state/graph/policies.go

@@ -10,8 +10,8 @@ import (
 	"sigs.k8s.io/controller-runtime/pkg/client"
 	v1 "sigs.k8s.io/gateway-api/apis/v1"
 
+	"github.com/nginx/nginx-gateway-fabric/v2/internal/controller/ngfsort"
 	"github.com/nginx/nginx-gateway-fabric/v2/internal/controller/nginx/config/policies"
-	ngfsort "github.com/nginx/nginx-gateway-fabric/v2/internal/controller/sort"
 	"github.com/nginx/nginx-gateway-fabric/v2/internal/controller/state/conditions"
 	"github.com/nginx/nginx-gateway-fabric/v2/internal/controller/state/validation"
 	"github.com/nginx/nginx-gateway-fabric/v2/internal/framework/kinds"

internal/controller/state/graph/route_common.go

@@ -14,7 +14,7 @@ import (
 	v1 "sigs.k8s.io/gateway-api/apis/v1"
 	v1alpha "sigs.k8s.io/gateway-api/apis/v1alpha2"
 
-	ngfSort "github.com/nginx/nginx-gateway-fabric/v2/internal/controller/sort"
+	"github.com/nginx/nginx-gateway-fabric/v2/internal/controller/ngfsort"
 	"github.com/nginx/nginx-gateway-fabric/v2/internal/controller/state/conditions"
 	"github.com/nginx/nginx-gateway-fabric/v2/internal/controller/state/validation"
 	"github.com/nginx/nginx-gateway-fabric/v2/internal/framework/kinds"
@@ -438,7 +438,7 @@ func bindRoutesToListeners(
 
 		// Sort the slice by timestamp and name so that we process the routes in the priority order
 		sort.Slice(l4RouteSlice, func(i, j int) bool {
-			return ngfSort.LessClientObject(l4RouteSlice[i].Source, l4RouteSlice[j].Source)
+			return ngfsort.LessClientObject(l4RouteSlice[i].Source, l4RouteSlice[j].Source)
 		})
 
 		// portHostnamesMap exists to detect duplicate hostnames on the same port