golangci-lint: ignore QF1010 (convert byte slices to strings before printing)

Printing a []byte and a string([]byte) are not the same thing. We typically print
bytes on purpose, so we don't need this linter.

Fixed one case where we should've been using t.Log instead of fmt.Println anyway.

Signed-off-by: Timo Beckers <[email protected]>

Author: ti-mo

.golangci.yaml

@@ -9,6 +9,14 @@ linters:
     - staticcheck
     - unused
   settings:
+    staticcheck:
+      checks:
+        [
+          # Defaults
+          "all", "-ST1000", "-ST1003", "-ST1016", "-ST1020", "-ST1021", "-ST1022",
+          # Convert slice of bytes to string when printing it.
+          "-QF1010",
+        ]
     depguard:
       rules:
         no-x-sys-unix:

btf/types_test.go

@@ -435,7 +435,7 @@ func TestInflateLegacyBitfield(t *testing.T) {
 		t.Run(test.name, func(t *testing.T) {
 			types, err := readAndInflateTypes(test.reader, binary.LittleEndian, 2, emptyStrings, nil)
 			if err != nil {
-				fmt.Println(before.Bytes())
+				t.Log(before.Bytes())
 				t.Fatal(err)
 			}