chore: adopt golangci-lint v2

Author: ReneWerner87

.github/workflows/golangci-lint.yml

@@ -38,6 +38,8 @@ jobs:
           cache: false
       - name: golangci-lint
         uses: golangci/golangci-lint-action@v6
+        env:
+          GOLANGCI_LINT_SKIP_GOVERSION_CHECK: "1"
         with:
           # NOTE: Keep this in sync with the version from .golangci.yml
-          version: 'v1.56.2'
+          version: 'v2.5.0'

.golangci.yml

@@ -1,5 +1,5 @@
-# Created based on v1.56.1
-# NOTE: Keep this in sync with the version in .github/workflows/linter.yml
+# Created for golangci-lint v2.5.0
+# NOTE: Keep this in sync with the version in .github/workflows/golangci-lint.yml
 
 version: 2
 
@@ -38,10 +38,6 @@ linters-settings:
     disabled-checks:
       - ifElseChain
 
-  gofumpt:
-    module-path: github.com/gofiber/template
-    extra-rules: true
-
   gosec:
     excludes:
       - G104 # Provided by errcheck
@@ -131,22 +127,11 @@ linters-settings:
       - name: use-any # TODO Enable for v3 release
         disabled: true
 
-  stylecheck:
-    checks:
-      - all
-      - -ST1000
-      - -ST1020
-      - -ST1021
-      - -ST1022
-
   tagliatelle:
     case:
       rules:
         json: snake
 
-  tenv:
-    all: true
-
   #unparam:
   #  check-exported: true
 
@@ -155,6 +140,15 @@ linters-settings:
       - github.com/gofiber/fiber/*
       - github.com/valyala/fasthttp
 
+formatters:
+  enable:
+    - gofumpt
+    - goimports
+  settings:
+    gofumpt:
+      module-path: github.com/gofiber/template
+      extra-rules: true
+
 issues:
    exclude-use-default: false
    exclude-dirs-use-default: false
@@ -185,13 +179,9 @@ linters:
     - gochecksumtype
     - goconst
     - gocritic
-    - gofmt
-    - gofumpt
-    - goimports
     - gomoddirectives
     - goprintffuncname
     - gosec
-    - gosimple
     - gosmopolitan
     - govet
     - grouper
@@ -214,7 +204,6 @@ linters:
     - rowserrcheck
     - sqlclosecheck
     - staticcheck
-    - stylecheck
     - tagalign
     - tagliatelle
     - testifylint
@@ -227,5 +216,5 @@ linters:
     - usestdlibvars
     - wastedassign
     - whitespace
+    - unqueryvet
     - wrapcheck
-    - tenv

template.go

@@ -1,3 +1,4 @@
+// Package template provides shared rendering primitives for Fiber template engines.
 package template
 
 import (
@@ -113,8 +114,8 @@ func (e *Engine) Reload(enabled bool) IEngineCore {
 	return e
 }
 
-// Check if the engine should reload the templates before rendering
-// Explicit Mute Unlock vs defer offers better performance
+// PreRenderCheck determines if the engine should reload the templates before rendering.
+// Explicit mutex unlock vs defer offers better performance.
 func (e *Engine) PreRenderCheck() bool {
 	e.Mutex.Lock()