Bumb to go 1.24 (#166)

* Bumb to go 1.24

Signed-off-by: Friedrich Gonzalez <1517449+friedrichg@users.noreply.github.com>

* Upgrade golangci-lint

like #159

Signed-off-by: Friedrich Gonzalez <1517449+friedrichg@users.noreply.github.com>

* Convert to switch

Signed-off-by: Friedrich Gonzalez <1517449+friedrichg@users.noreply.github.com>

---------

Signed-off-by: Friedrich Gonzalez <1517449+friedrichg@users.noreply.github.com>

Author: friedrichg

.github/workflows/ci.yml

@@ -32,4 +32,4 @@ jobs:
           go-version-file: .go-version
           cache: true
       - name: Golangci-lint
-        uses: golangci/golangci-lint-action@v6.5.1
+        uses: golangci/golangci-lint-action@v8.0.0

.go-version

@@ -1 +1 @@
-1.22
+1.24

.golangci.yml

@@ -1,11 +1,29 @@
+version: "2"
 linters:
   enable:
-    - goimports
     - misspell
     - revive
-
-issues:
-  exclude-rules:
-    - path: _test.go
-      linters:
-        - errcheck
+  exclusions:
+    generated: lax
+    presets:
+      - comments
+      - common-false-positives
+      - legacy
+      - std-error-handling
+    rules:
+      - linters:
+          - errcheck
+        path: _test.go
+    paths:
+      - third_party$
+      - builtin$
+      - examples$
+formatters:
+  enable:
+    - goimports
+  exclusions:
+    generated: lax
+    paths:
+      - third_party$
+      - builtin$
+      - examples$

walk.go

@@ -356,13 +356,14 @@ func (s *PromQLSmith) walkFunctions(expr *parser.Call, depth int) {
 	}
 
 	expr.Args = make([]parser.Expr, len(expr.Func.ArgTypes))
-	if expr.Func.Name == "holt_winters" {
+	switch expr.Func.Name {
+	case "holt_winters":
 		s.walkHoltWinters(expr, depth)
 		return
-	} else if expr.Func.Name == "label_replace" {
+	case "label_replace":
 		s.walkLabelReplace(expr, depth)
 		return
-	} else if expr.Func.Name == "info" {
+	case "info":
 		s.walkInfo(expr, depth)
 		return
 	}