Bump golangci-lint to v1.59

... and fix issues reported by the new version.

Signed-off-by: James Alseth <[email protected]>

Author: jalseth

.github/workflows/pull_request.yaml

@@ -17,7 +17,7 @@ jobs:
       - name: lint go
         uses: golangci/golangci-lint-action@v6
         with:
-          version: v1.52
+          version: v1.59
           args: --timeout=5m --color=always --max-same-issues=0 --max-issues-per-linter=0
 
   acceptance:

.golangci.yaml

@@ -6,7 +6,6 @@ linter-settings:
 linters:
   disable-all: true
   enable:
-    - deadcode
     - errcheck
     - goconst
     - gofmt
@@ -20,10 +19,8 @@ linters:
     - predeclared
     - revive
     - staticcheck
-    - structcheck
     - typecheck
     - unconvert
     - unparam
     - unused
-    - varcheck
     - wastedassign

internal/commands/convert.go

@@ -20,7 +20,7 @@ func newConvertCommand() *cobra.Command {
 
 konstraint convert examples`,
 
-		RunE: func(cmd *cobra.Command, args []string) error {
+		RunE: func(_ *cobra.Command, args []string) error {
 			path := "."
 			if len(args) > 0 {
 				path = args[0]

internal/rego/rego.go

@@ -657,17 +657,19 @@ func (r Rego) SkipConstraint() bool {
 
 func parseDirectory(directory string, parseImports bool) ([]Rego, error) {
 	// Recursively find all rego files (ignoring test files), starting at the given directory.
-	result, err := loader.NewFileLoader().WithProcessAnnotation(true).Filtered([]string{directory}, func(abspath string, info os.FileInfo, depth int) bool {
-		if strings.HasSuffix(info.Name(), "_test.rego") {
-			return true
-		}
+	result, err := loader.NewFileLoader().
+		WithProcessAnnotation(true).
+		Filtered([]string{directory}, func(_ string, info os.FileInfo, _ int) bool {
+			if strings.HasSuffix(info.Name(), "_test.rego") {
+				return true
+			}
 
-		if !info.IsDir() && filepath.Ext(info.Name()) != ".rego" {
-			return true
-		}
+			if !info.IsDir() && filepath.Ext(info.Name()) != ".rego" {
+				return true
+			}
 
-		return false
-	})
+			return false
+		})
 	if err != nil {
 		return nil, fmt.Errorf("filter rego files: %w", err)
 	}