ci: update golangci-lint configuration (#40)

.github/workflows/golangci-lint.yml

@@ -1,9 +1,15 @@
 name: golangci-lint
+
 on:
   push:
     branches:
-      - master
+      - '**'
   pull_request:
+    branches:
+      - master
+
+permissions:
+  contents: read
 
 jobs:
   golangci:
@@ -13,5 +19,16 @@ jobs:
       - name: Checkout code
         uses: actions/checkout@v4
 
-      - name: golangci-lint
-        uses: golangci/golangci-lint-action@v4
+      - name: Get go version from go.mod
+        run: |
+          echo "GO_VERSION=$(grep '^go ' go.mod | cut -d " " -f 2)" >> $GITHUB_ENV
+
+      - name: Setup-go
+        uses: actions/setup-go@v5
+        with:
+          go-version: ${{ env.GO_VERSION }}
+
+      - name: Run golangci-lint
+        uses: golangci/golangci-lint-action@v6
+        with:
+          version: v1.60.3

.golangci.yml

@@ -1,33 +1,53 @@
 run:
-  skip-dirs:
-    - benchmarks
+  timeout: 2m
 
 linters:
   disable-all: true
   enable:
     - dupl
     - errcheck
-    - exportloopref
+    - errname
+    - errorlint
     - funlen
+    - gci
     - goconst
     - gocritic
     - gocyclo
     - gofmt
     - goimports
+    - gosec
     - gosimple
     - govet
     - ineffassign
     - lll
     - misspell
+    - nolintlint
     - prealloc
     - revive
     - staticcheck
+    - stylecheck
+    - thelper
+    - tparallel
     - typecheck
     - unconvert
+    - unparam
+    - unused
+    - whitespace
+
+linters-settings:
+  gosec:
+    excludes:
+      - G115
+  thelper:
+    test:
+      begin: false
 
 issues:
   exclude-rules:
-    # Exclude some linters from running on tests files.
     - path: _test\.go
       linters:
         - errcheck
+        - funlen
+
+  exclude-dirs:
+    - benchmarks

value_test.go

@@ -7,7 +7,6 @@ import (
 	"github.com/reugn/async/internal/util"
 )
 
-//nolint:funlen
 func TestValueCompareAndSwap(t *testing.T) {
 	var value Value
 	swapped := value.CompareAndSwap(1, 2)