chore: update go, mods, workflow & linter

.github/workflows/lint.yaml

@@ -0,0 +1,39 @@
+name: golangci
+
+on:
+  pull_request:
+
+jobs:
+
+  main:
+    name: lint
+    runs-on: ubuntu-latest
+    env:
+      GOLANGCI_LINT_VERSION: v1.60
+      CGO_ENABLED: 0
+
+    steps:
+      # https://github.com/marketplace/actions/checkout
+      - name: Check out code
+        uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
+
+      # https://github.com/marketplace/actions/setup-go-environment
+      - name: Set up Go ${{ env.GO_VERSION }}
+        uses: actions/setup-go@v5
+        with:
+          go-version-file: 'go.mod'
+
+      - name: Check and get dependencies
+        run: |
+          go mod tidy
+          git diff --exit-code go.mod
+          git diff --exit-code go.sum
+          go mod download
+
+      # https://github.com/marketplace/actions/golangci-lint
+      - name: golangci-lint
+        uses: golangci/golangci-lint-action@v6
+        with:
+          version: "${{ env.GOLANGCI_LINT_VERSION }}"

.github/workflows/test.yaml

@@ -0,0 +1,31 @@
+name: go
+
+on:
+  pull_request:
+
+jobs:
+
+  cross:
+    name: test
+    runs-on: ${{ matrix.os }}
+    env:
+      CGO_ENABLED: 0
+
+    strategy:
+      matrix:
+        os: [ubuntu-latest, macos-latest, windows-latest]
+
+    steps:
+      # https://github.com/marketplace/actions/checkout
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      # https://github.com/marketplace/actions/setup-go-environment
+      - name: Set up Go ${{ matrix.go-version }}
+        uses: actions/setup-go@v5
+        with:
+          go-version-file: 'go.mod'
+
+      - name: Test
+        run: go test -v -cover ./...
+

.golangci.yml

@@ -1,11 +1,14 @@
 run:
   timeout: 2m
-  skip-files:
-    - ^unsecured/
 
 linters-settings:
+  depguard:
+    rules:
+      main:
+        deny:
+          - pkg: "github.com/pkg/errors"
+            desc: Should be replaced by standard lib errors package
   govet:
-    check-shadowing: true
     enable-all: true
     disable:
       - fieldalignment
@@ -31,24 +34,18 @@ linters-settings:
 linters:
   enable-all: true
   disable:
-    - maligned # Deprecated
-    - scopelint # Deprecated
-    - golint # Deprecated
-    - interfacer # Deprecated
-    - exhaustivestruct # Deprecated
-    - ifshort # Deprecated
-    - varcheck # Deprecated
-    - deadcode # Deprecated
-    - structcheck # Deprecated
-    - nosnakecase # Deprecated
+    - execinquery # Deprecated
+    - gomnd # Deprecated
     - cyclop # duplicate of gocyclo
     - rowserrcheck # SQL
     - sqlclosecheck # SQL
     - dupl
     - lll
     - nestif
-    - gomnd
-    - goerr113
+    - mnd
+    - err113
+    - tagalign
+    - prealloc
     - wrapcheck
     - nlreturn
     - wsl
@@ -62,7 +59,7 @@ linters:
 
 issues:
   exclude-use-default: false
-  max-per-linter: 0
+  max-issues-per-linter: 0
   max-same-issues: 0
   exclude:
     - 'ST1000: at least one file in a package should have a package comment'
@@ -74,3 +71,6 @@ issues:
         - goconst
         - gosec
         - maintidx
+    - path: 'parser/element_fill.go'
+      linters:
+        - mnd

Makefile

@@ -1,7 +1,5 @@
 .PHONY: clean lint lint-fix test
 
-export GO111MODULE=on
-
 default: lint test
 
 test:
@@ -11,4 +9,4 @@ lint:
 	golangci-lint run
 
 lint-fix:
-	golangci-lint run --fix
+	golangci-lint run --fix

cli/commands_test.go

@@ -39,7 +39,6 @@ func TestCommand_AddCommand(t *testing.T) {
 	}
 
 	for _, test := range testCases {
-		test := test
 		t.Run(test.desc, func(t *testing.T) {
 			t.Parallel()
 
@@ -80,7 +79,7 @@ func TestCommand_PrintHelp(t *testing.T) {
 						Field string
 					}
 				}{},
-				Run: func(args []string) error {
+				Run: func(_ []string) error {
 					return nil
 				},
 				CustomHelpFunc: func(w io.Writer, _ *Command) error {
@@ -102,7 +101,6 @@ func TestCommand_PrintHelp(t *testing.T) {
 	}
 
 	for _, test := range testCases {
-		test := test
 		t.Run(test.desc, func(t *testing.T) {
 			t.Parallel()
 
@@ -740,7 +738,7 @@ func Test_execute_configuration(t *testing.T) {
 		Description:   "sub1",
 		Configuration: element,
 		Resources:     []ResourceLoader{&FlagLoader{}},
-		Run: func(args []string) error {
+		Run: func(_ []string) error {
 			return nil
 		},
 	}
@@ -804,7 +802,7 @@ func Test_execute_configuration_file(t *testing.T) {
 				Description:   "sub1",
 				Configuration: element,
 				Resources:     []ResourceLoader{fileLoader, &FlagLoader{}},
-				Run: func(args []string) error {
+				Run: func(_ []string) error {
 					return nil
 				},
 			}
@@ -837,7 +835,7 @@ func Test_execute_help(t *testing.T) {
 		Name:          "root",
 		Description:   "Description for root",
 		Configuration: element,
-		Run: func(args []string) error {
+		Run: func(_ []string) error {
 			return nil
 		},
 	}

cli/file_finder_test.go

@@ -144,7 +144,6 @@ func TestFinder_getPaths(t *testing.T) {
 	}
 
 	for _, test := range testCases {
-		test := test
 		t.Run(test.desc, func(t *testing.T) {
 			t.Parallel()
 

cli/help_test.go

@@ -25,7 +25,7 @@ func TestPrintHelp(t *testing.T) {
 					Name:          "root",
 					Description:   "Description for root",
 					Configuration: element,
-					Run: func(args []string) error {
+					Run: func(_ []string) error {
 						return nil
 					},
 				}
@@ -85,7 +85,7 @@ Flags:
 					Name:          "sub1",
 					Description:   "Description for sub1",
 					Configuration: element,
-					Run: func(args []string) error {
+					Run: func(_ []string) error {
 						return nil
 					},
 				})
@@ -95,7 +95,7 @@ Flags:
 					Name:          "sub2",
 					Description:   "Description for sub2",
 					Configuration: element,
-					Run: func(args []string) error {
+					Run: func(_ []string) error {
 						return nil
 					},
 				})
@@ -149,7 +149,7 @@ Flags:
 					Name:          "root",
 					Description:   "Description for root",
 					Configuration: nil,
-					Run: func(args []string) error {
+					Run: func(_ []string) error {
 						return nil
 					},
 				}
@@ -173,7 +173,7 @@ Use "root [command] --help" for help on any command.
 							Field string
 						}
 					}{},
-					Run: func(args []string) error {
+					Run: func(_ []string) error {
 						return nil
 					},
 				}
@@ -197,7 +197,6 @@ Flags:
 	}
 
 	for _, test := range testCases {
-		test := test
 		t.Run(test.desc, func(t *testing.T) {
 			t.Parallel()
 

env/env_test.go

@@ -169,7 +169,6 @@ func TestDecode(t *testing.T) {
 	}
 
 	for _, test := range testCases {
-		test := test
 		t.Run(test.desc, func(t *testing.T) {
 			t.Parallel()