chore: update golangci-lint config to v2

Signed-off-by: Francesco Canovai <francesco.canovai@enterprisedb.com>

.golangci.yml

@@ -1,45 +1,25 @@
-linters-settings:
+version: "2"
-  lll:
-    line-length: 120
-  gci:
-    sections:
-      - standard
-      - default
-      - prefix(github.com/cloudnative-pg/plugin-barman-cloud)
-      - blank
-      - dot
-  gosec:
-    excludes:
-      - G101 # remove this exclude when https://github.com/securego/gosec/issues/1001 is fixed
-
 linters:
-  # please, do not use `enable-all`: it's deprecated and will be removed soon.
+  default: none
-  # inverted configuration with `enable-all` and `disable` is not scalable during updates of golangci-lint
-  disable-all: true
   enable:
     - asciicheck
     - bodyclose
+    - copyloopvar
     - dogsled
     - dupl
     - durationcheck
     - errcheck
-    - copyloopvar
+    - ginkgolinter
-    - gci
     - gocognit
     - goconst
     - gocritic
     - gocyclo
-    - gofmt
-    - gofumpt
     - goheader
-    - goimports
     - gomoddirectives
     - gomodguard
     - goprintffuncname
     - gosec
-    - gosimple
     - govet
-    - ginkgolinter
     - importas
     - ineffassign
     - lll
@@ -53,82 +33,60 @@ linters:
     - rowserrcheck
     - sqlclosecheck
     - staticcheck
-    - stylecheck
     - thelper
     - tparallel
-    - typecheck
     - unconvert
     - unparam
     - unused
     - wastedassign
     - whitespace
-
+  settings:
-  # to be checked:
+    lll:
-  # - errorlint
+      line-length: 120
-  # - forbidigo
+  exclusions:
-  # - forcetypeassert
+    generated: lax
-  # - goerr113
+    rules:
-  # - ifshort
+      - linters:
-  # - nilerr
-  # - nlreturn
-  # - noctx
-  # - nolintlint
-  # - paralleltest
-  # - promlinter
-  # - tagliatelle
-  # - wrapcheck
-
-  # don't enable:
-  # - cyclop
-  # - depguard
-  # - exhaustive
-  # - exhaustivestruct
-  # - funlen
-  # - gochecknoglobals
-  # - gochecknoinits
-  # - godot
-  # - godox
-  # - gomnd
-  # - testpackage
-  # - wsl
-
-  # deprecated:
-  # - deadcode
-  # - golint
-  # - interfacer
-  # - maligned
-  # - scopelint
-  # - structcheck
-  # - varcheck
-
-run:
-  timeout: 10m
-
-issues:
-  exclude-rules:
-    # Allow dot imports for ginkgo and gomega
-    - source: ginkgo|gomega
-      linters:
           - revive
-      text: "should not use dot imports"
+        text: should not use dot imports
-    # Exclude some linters from running on tests files.
+        source: ginkgo|gomega
-    - path: _test\.go
+      - linters:
-      linters:
           - goconst
-    # Exclude lll issues for lines with long annotations
+        path: _test\.go
       - linters:
           - lll
-      source: "//\\s*\\+"
+        source: //\s*\+
-    # We have no control of this in zz_generated files and it looks like that excluding those files is not enough
-    # so we disable "ST1016: methods on the same type should have the same receiver name" in api directory
       - linters:
-      - stylecheck
+          - staticcheck
-      text: "ST1016:"
         path: api/
-  exclude-use-default: false
+        text: 'ST1016:'
-  exclude-dirs:
+    paths:
-    # This tests are generated by kubebuilder
-    - test
-  exclude-files:
       - zz_generated.*
       - internal/operator/controller/suite_test.go
+      - test
+      - third_party$
+      - builtin$
+      - examples$
+formatters:
+  enable:
+    - gci
+    - gofmt
+    - gofumpt
+    - goimports
+  settings:
+    gci:
+      sections:
+        - standard
+        - default
+        - prefix(github.com/cloudnative-pg/plugin-barman-cloud)
+        - blank
+        - dot
+  exclusions:
+    generated: lax
+    paths:
+      - zz_generated.*
+      - internal/operator/controller/suite_test.go
+      - test
+      - third_party$
+      - builtin$
+      - examples$