From e9031d97bda11221f74a490990d10d029a0ae487 Mon Sep 17 00:00:00 2001
From: Panagiotis Georgiadis <pgeorgia@redhat.com>
Date: Thu, 25 Mar 2021 00:51:10 +0100
Subject: [PATCH] Disable fieldalignment

---
 .golangci.yml | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/.golangci.yml b/.golangci.yml
index c2e5d155..f8d9e85d 100644
--- a/.golangci.yml
+++ b/.golangci.yml
@@ -18,15 +18,20 @@ linters-settings:
     disabled-checks:
   gocyclo:
     min-complexity: 15
+  gofmt:
+    simplify: true
   goimports:
     local-prefixes: github.com/OpenDiablo2/OpenDiablo2
   golint:
     min-confidence: 0.8
   govet:
-    enable:
-      - fieldalignment
+    enable-all: true
     check-shadowing: true
-
+    disable:
+      # While struct sizes could be smaller if fields aligned properly, that also leads
+      # to possibly non-intuitive layout of struct fields (harder to read). Disable
+      # `fieldalignment` check here until we evaluate if it is worthwhile.
+      - fieldalignment
   lll:
     line-length: 140
   misspell: