Enable formatters as well as linters

Currently gofmt and goimports have separate targets,
which means that they are forgotten for new changes.

Author: afbjorklund

.golangci.yaml

@@ -170,3 +170,10 @@ linters:
       - path: '(.+)\.go$'
         text: "Error return value of `.*` is not checked"
         linter: errcheck
+formatters:
+  enable:
+    - gofmt
+    - goimports
+  exclusions:
+    paths:
+      - third_party

cmd/minikube/cmd/config/view.go

@@ -40,7 +40,7 @@ type ViewTemplate struct {
 var configViewCmd = &cobra.Command{
 	Use:   "view",
 	Short: "Display values currently set in the minikube config file",
-	Long:  `Display values currently set in the minikube config file. 
+	Long: `Display values currently set in the minikube config file.
 	The output format can be customized using the --format flag, which accepts a Go template. 
 	The config file is typically located at "~/.minikube/config/config.json".`,
 	Run: func(_ *cobra.Command, _ []string) {

cmd/minikube/cmd/start.go

@@ -544,7 +544,7 @@ func updateDriver(driverName string) {
 		}
 		if errors.Is(err, auxdriver.ErrAuxDriverVersionNotinPath) {
 			exit.Error(reason.DrvAuxNotHealthy, "Aux driver"+driverName, err)
-		} //if failed to update but not a fatal error, log it and continue (old version might still work)
+		} // if failed to update but not a fatal error, log it and continue (old version might still work)
 		out.WarningT("Unable to update {{.driver}} driver: {{.error}}", out.V{"driver": driverName, "error": err})
 	}
 }

pkg/minikube/assets/addons_test.go

@@ -45,7 +45,7 @@ func mapsEqual(a, b map[string]string) bool {
 
 func TestParseMapString(t *testing.T) {
 	cases := map[string]map[string]string{
-		"Aardvark=1,B=2,Cantaloupe=3":         {"Aardvark": "1", "B": "2", "Cantaloupe": "3"},
+		"Aardvark=1,B=2,Cantaloupe=3":        {"Aardvark": "1", "B": "2", "Cantaloupe": "3"},
 		"A=,B=2,C=":                          {"A": "", "B": "2", "C": ""},
 		"":                                   {},
 		"malformed,good=howdy,manyequals==,": {"good": "howdy"},

pkg/minikube/assets/vm_assets.go

@@ -20,13 +20,13 @@ import (
 	"bytes"
 	"embed"
 	"fmt"
-	"text/template"
 	"io"
 	"os"
 	"path"
 	"path/filepath"
 	"strconv"
 	"strings"
+	"text/template"
 	"time"
 
 	"github.com/pkg/errors"

pkg/minikube/download/licenses.go

@@ -67,15 +67,15 @@ func Licenses(dir string) error {
 		return fmt.Errorf("failed to create temporary file: %v", err)
 	}
 	defer func() {
-                if err := os.Remove(tempFile.Name()); err != nil {
-                        klog.Warningf("Failed to remove temp file %s: %v", tempFile.Name(), err)
-                }
-        }()
+		if err := os.Remove(tempFile.Name()); err != nil {
+			klog.Warningf("Failed to remove temp file %s: %v", tempFile.Name(), err)
+		}
+	}()
 	defer func() {
-                if err := tempFile.Close(); err != nil {
-                        klog.Warningf("Failed to close temp file %s: %v", tempFile.Name(), err)
-                }
-        }()
+		if err := tempFile.Close(); err != nil {
+			klog.Warningf("Failed to close temp file %s: %v", tempFile.Name(), err)
+		}
+	}()
 
 	if _, err := io.Copy(tempFile, resp.Body); err != nil {
 		return fmt.Errorf("failed to copy downloaded content from %s: %v", url, err)

pkg/minikube/tunnel/route_windows_test.go

@@ -139,11 +139,13 @@ Persistent Routes:
 	expectedRt := routingTable{
 		routingTableLine{
 			route: unsafeParseRoute("127.0.0.1", "10.96.0.0/12"),
-			line:  " 	    10.96.0.0      255.240.0.0        127.0.0.1        127.0.0.1    281",
+			line: " 	    10.96.0.0      255.240.0.0" +
+				"        127.0.0.1        127.0.0.1    281",
 		},
 		routingTableLine{
 			route: unsafeParseRoute("192.168.1.2", "10.211.55.0/24"),
-			line:  "      10.211.55.0    255.255.255.0      192.168.1.2      10.211.55.3    281",
+			line: "      10.211.55.0    255.255.255.0" +
+				"      192.168.1.2      10.211.55.3    281",
 		},
 	}
 	if !reflect.DeepEqual(rt.String(), expectedRt.String()) {