build(deps): bump golangci/golangci-lint-action from 6 to 7 (#128)

* build(deps): bump golangci/golangci-lint-action from 6 to 7

Bumps [golangci/golangci-lint-action](https://github.com/golangci/golangci-lint-action) from 6 to 7.
- [Release notes](https://github.com/golangci/golangci-lint-action/releases)
- [Commits](golangci/golangci-lint-action@v6...v7)


---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Lorenz Kästle <[email protected]>
Co-authored-by: Markus Opolka <[email protected]>

Author: 3 people

.github/workflows/golangci-lint.yml

@@ -13,6 +13,6 @@ jobs:
       - uses: actions/checkout@v4
 
       - name: golangci-lint
-        uses: golangci/golangci-lint-action@v6
+        uses: golangci/golangci-lint-action@v7
         with:
-          version: v1.61
+          version: v2.0.2

.golangci.yml

@@ -1,43 +1,54 @@
+version: "2"
 run:
-  timeout: 5m
   tests: false
-issues.exclude-files:
-    - 'testhelper/for_main.go'
-issues:
-  exclude-rules:
-    - path: 'result/overall.go'
-      linters:
-        - nestif
-    - path: 'convert/bytes_common.go'
-      linters:
-        - ireturn
 linters:
-   enable-all: true
-   disable:
-     - cyclop
-     - depguard
-     - exhaustruct
-     - forbidigo
-     - forcetypeassert
-     - gci
-     - gochecknoglobals
-     - gochecknoinits
-     - godox
-     - godot
-     - err113
-     - gofumpt
-     - gomnd
-     - lll
-     - musttag
-     - nakedret
-     - nlreturn
-     - nolintlint
-     - nonamedreturns
-     - tagliatelle
-     - varnamelen
-     - wrapcheck
-linters-settings:
-   estif:
-     min-complexity: 4
-   maligned:
-     suggest-new: true
+  default: all
+  disable:
+    - cyclop
+    - depguard
+    - err113
+    - exhaustruct
+    - forbidigo
+    - forcetypeassert
+    - gochecknoglobals
+    - gochecknoinits
+    - godot
+    - godox
+    - lll
+    - mnd
+    - musttag
+    - nakedret
+    - nlreturn
+    - nolintlint
+    - nonamedreturns
+    - tagliatelle
+    - varnamelen
+    - wrapcheck
+  exclusions:
+    generated: lax
+    presets:
+      - comments
+      - common-false-positives
+      - legacy
+      - std-error-handling
+    rules:
+      - linters:
+          - nestif
+        path: result/overall.go
+      - linters:
+          - ireturn
+        path: convert/bytes_common.go
+    paths:
+      - third_party$
+      - builtin$
+      - examples/
+formatters:
+  enable:
+    - gofmt
+    - goimports
+  exclusions:
+    generated: lax
+    paths:
+      - third_party$
+      - builtin$
+      - examples$

convert/bytes_common.go

@@ -23,7 +23,8 @@ func ParseBytes(value string) (ByteAny, error) {
 	value = strings.TrimSpace(value)
 
 	// Split number and unit by first non-numeric rune
-	firstNonNumeric := func(c rune) bool { return !(c >= '0' && c <= '9' || c == '.') }
+	firstNonNumeric := func(c rune) bool { return !(c >= '0' && c <= '9' || c == '.') } //nolint: staticcheck
+
 	i := strings.IndexFunc(value, firstNonNumeric)
 
 	var unit string

perfdata/list.go

@@ -9,10 +9,10 @@ import (
 type PerfdataList []*Perfdata
 
 // String returns string representations of all Perfdata
-func (l PerfdataList) String() string {
+func (l *PerfdataList) String() string {
 	var out strings.Builder
 
-	for _, p := range l {
+	for _, p := range *l {
 		pfDataString, err := p.ValidatedString()
 
 		// Ignore perfdata points which fail to format

perfdata/list_test.go

@@ -15,7 +15,7 @@ func ExamplePerfdataList() {
 	fmt.Println(list)
 
 	// Output:
-	// test1=23 test2=42
+	// [test1=23 test2=42]
 }
 
 func TestPerfdataListFormating(t *testing.T) {

testhelper/for_main.go

@@ -15,7 +15,7 @@ func RunMainTest(f func(), args ...string) string {
 	base := []string{"check_with_go_test"}
 	origArgs := os.Args
 
-	os.Args = append(base, args...)
+	os.Args = append(base, args...) //nolint: gocritic
 	stdout := CaptureStdout(f)
 	os.Args = origArgs