diff --git a/.golangci.yaml b/.golangci.yaml index 6070081465..0c46fa8212 100644 --- a/.golangci.yaml +++ b/.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 diff --git a/cmd/minikube/cmd/config/view.go b/cmd/minikube/cmd/config/view.go index 298d1e1553..836aef11fc 100644 --- a/cmd/minikube/cmd/config/view.go +++ b/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) { diff --git a/cmd/minikube/cmd/start.go b/cmd/minikube/cmd/start.go index 4aaa307d56..8934f7ccc4 100644 --- a/cmd/minikube/cmd/start.go +++ b/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}) } } diff --git a/pkg/minikube/assets/addons_test.go b/pkg/minikube/assets/addons_test.go index de5c995ab1..865c3cc4e7 100644 --- a/pkg/minikube/assets/addons_test.go +++ b/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"}, diff --git a/pkg/minikube/assets/vm_assets.go b/pkg/minikube/assets/vm_assets.go index f539a94882..b093fb6f04 100644 --- a/pkg/minikube/assets/vm_assets.go +++ b/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" diff --git a/pkg/minikube/download/licenses.go b/pkg/minikube/download/licenses.go index bcda1f6d29..977f569015 100644 --- a/pkg/minikube/download/licenses.go +++ b/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) diff --git a/pkg/minikube/tunnel/route_windows_test.go b/pkg/minikube/tunnel/route_windows_test.go index 88843e4cb7..da42fdfe30 100644 --- a/pkg/minikube/tunnel/route_windows_test.go +++ b/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()) {