Bump golangci/golangci-lint from v2.4.0 to v2.5.0 (#173)

* Bump golangci/golangci-lint from v2.4.0 to v2.5.0

Bumps golangci/golangci-lint from v2.4.0 to v2.5.0.

---
updated-dependencies:
- dependency-name: golangci/golangci-lint
  dependency-version: v2.5.0
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

* Update config

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Marat Reymers <marat.maratori@gmail.com>

Author: dependabot[bot]

.github/workflows/ci.yml

@@ -55,7 +55,7 @@ jobs:
           go-version: "1.25.1" # update together with dev.dockerfile
       - uses: golangci/golangci-lint-action@v8
         with:
-          version: "v2.4.0" # update together with dev.dockerfile
+          version: "v2.5.0" # update together with dev.dockerfile
 
   lint-latest-deps:
     runs-on: ubuntu-latest
@@ -67,7 +67,7 @@ jobs:
       - run: make apply-latest-deps
       - uses: golangci/golangci-lint-action@v8
         with:
-          version: "v2.4.0" # update together with dev.dockerfile
+          version: "v2.5.0" # update together with dev.dockerfile
 
   check-tidy:
     name: go mod tidy

.golangci.yml

@@ -64,6 +64,7 @@ linters:
     - goconst # finds repeated strings that could be replaced by a constant
     - gocritic # provides diagnostics that check for bugs, performance and style issues
     - gocyclo # computes and checks the cyclomatic complexity of functions
+    - godoclint # checks Golang's documentation practice
     - godot # checks if comments end in a period
     - gomoddirectives # manages the use of 'replace', 'retract', and 'excludes' directives in go.mod
     - goprintffuncname # checks that printf-like functions are named with f at the end
@@ -72,6 +73,7 @@ linters:
     - iface # checks the incorrect use of interfaces, helping developers avoid interface pollution
     - ineffassign # detects when assignments to existing variables are not used
     - intrange # finds places where for loops could make use of an integer range
+    - iotamixing # checks if iotas are being used in const blocks with other non-iota declarations
     - loggercheck # checks key value pairs for common logger libraries (kitlog,klog,logr,zap)
     - makezero # finds slice declarations with non-zero initial length
     - mirror # reports wrong mirror patterns of bytes/strings usage
@@ -105,6 +107,7 @@ linters:
     - tparallel # detects inappropriate usage of t.Parallel() method in your Go test codes
     - unconvert # removes unnecessary type conversions
     - unparam # reports unused function parameters
+    - unqueryvet # detects SELECT * in SQL queries and SQL builders, encouraging explicit column selection
     - unused # checks for unused constants, variables, functions and types
     - usestdlibvars # detects the possibility to use variables/constants from the Go standard library
     - usetesting # reports uses of functions with replacement inside the testing package
@@ -300,6 +303,14 @@ linters:
           # Default: true
           skipRecvDeref: false
 
+    godoclint:
+      # List of rules to enable in addition to the default set.
+      # Default: empty
+      enable:
+        # Assert no unused link in godocs.
+        # https://github.com/godoc-lint/godoc-lint?tab=readme-ov-file#no-unused-link
+        - no-unused-link
+
     govet:
       # Enable all analyzers.
       # Default: false

dev.dockerfile

@@ -2,7 +2,7 @@
 FROM golang:1.25.1 AS go
 
 # update together with .github/workflows/ci.yml
-FROM golangci/golangci-lint:v2.4.0 AS linter
+FROM golangci/golangci-lint:v2.5.0 AS linter
 
 FROM go AS dev
 ENV INSIDE_DEV_CONTAINER=1