golangci-lint: switch to fieldalignment

fieldalignment replaces maligned. Errors related to pointer ordering (for GC) are ignored; see golang/go#44877 for details. Fixes: submariner-io#1252 Signed-off-by: Stephen Kitt <[email protected]>
skitt · May 5, 2021 · c324aca · c324aca
1 parent aa87534
commit c324aca
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/.golangci.yml b/.golangci.yml
@@ -11,10 +11,11 @@ linters-settings:
       - unnamedResult
   gocyclo:
     min-complexity: 20
+  govet:
+    enable:
+      - fieldalignment
   lll:
     line-length: 140
-  maligned:
-    suggest-new: true
 linters:
   disable-all: true
   enable:
@@ -39,7 +40,6 @@ linters:
     - ineffassign
     - interfacer
     - lll
-    - maligned
     - misspell
     - nakedret
     - prealloc
@@ -83,3 +83,9 @@ issues:
         - stylecheck
       text: "ST1001: "
       path: test
+
+    # Ignore pointer bytes in struct alignment tests (this is a very
+    # minor optimisation)
+    - linters:
+        - govet
+      text: "pointer bytes could be"