chore: update golangci-lint to latest v1 version and simplify config (#294)

* ci: upgrade to latest v1 version of `golangci-lint`

* chore: update linter config

* chore: sort disabled linters

* chore: enable all linters by default

* chore: disable unwanted linters

* chore: add back `prettier-ignore`

Author: G-Rath

.github/workflows/checks.yml

@@ -46,7 +46,7 @@ jobs:
       - name: Run golangci-lint
         uses: golangci/golangci-lint-action@55c2c1448f86e01eaae002a5a3a9624417608d84 # v6.5.2
         with:
-          version: v1.60
+          version: v1.64
   go-fmt:
     permissions:
       contents: read # to fetch code (actions/checkout)

.github/workflows/release.yml

@@ -45,7 +45,7 @@ jobs:
       - name: Run golangci-lint
         uses: golangci/golangci-lint-action@55c2c1448f86e01eaae002a5a3a9624417608d84 # v6.5.2
         with:
-          version: v1.60
+          version: v1.64
   go-fmt:
     permissions:
       contents: read # to fetch code (actions/checkout)

.golangci.yaml

@@ -1,35 +1,32 @@
 output:
   sort-results: true
 linters:
-  enable:
-    - gofmt
-    - goimports
+  enable-all: true
   # prettier-ignore
   disable:
-    - tagliatelle      # we're parsing data from external sources
-    - varnamelen       # maybe later
+    - cyclop
     - exhaustruct      # overkill
     - forcetypeassert  # too hard
-    - lll              # line length is hard
-    - godox            # to-do comments are fine
+    - funlen
+    - gocognit
+    - gocyclo
     - godot            # comments are fine without full stops
-    - mnd              # not every number is magic
-    - wsl              # disagree with, for now
+    - godox            # to-do comments are fine
+    - maintidx
+    - gofumpt
     - ireturn          # disagree with, sort of
+    - lll              # line length is hard
+    - mnd              # not every number is magic
+    - nestif
     - nonamedreturns   # they have their uses
+    - tagliatelle      # we're parsing data from external sources
+    - tenv             # Deprecated
+    - varnamelen       # maybe later
+    - wsl              # disagree with, for now
     - intrange
     - copyloopvar
     - predeclared
     - govet
-  presets:
-    - bugs
-    - comment
-    - error
-    - performance
-    - sql
-    - style
-    - test
-    - unused
 
 linters-settings:
   depguard:
@@ -44,6 +41,8 @@ linters-settings:
               'Use github.com/g-rath/osv-detector/internal/cachedregexp instead'
 
 issues:
+  max-issues-per-linter: 0
+  max-same-issues: 0
   exclude-rules:
     - path: _test\.go
       linters:

Makefile

@@ -21,7 +21,7 @@ test-with-coverage:
 lint:	lint-with-golangci-lint lint-with-go-fmt
 
 lint-with-golangci-lint:
-	go run github.com/golangci/golangci-lint/cmd/golangci-lint@v1.60.0 run ./... --max-same-issues 0
+	go run github.com/golangci/golangci-lint/cmd/golangci-lint@v1.64.8 run ./...
 
 lint-with-go-fmt:
 	gofmt -s -d */**.go