* fix QF1011: could omit type *os.File from declaration; it will be inferred from the right-hand side
* fix QF1012: Use fmt.Fprintf(x, ...) instead of x.Write(fmt.Sprintf(...))
* fix QF1001: could apply De Morgan's law
* fix QF1003: could use tagged switch
* fix weakCond: suspicious ; nil check may not be enough, check for len (gocritic)
* fix docStub: silencing go lint doc-comment warnings is unadvised
* fix builtinShadow: shadowing of predeclared identifier: error
* fix importShadow: shadow of imported package
* fix nestingReduce: invert if cond, replace body with , move old body after the statement
* useless-break: useless break in case clause (revive)
* Clear the redundant content in golangci.yaml file
This requires the following changes:
1. We need to pass in -ldflags to `go test`, since we need to know the curren iso version to correctly generate the description for --iso-url
2. We need to rewrite the description of the --driver flag on `minikube start` because it is operating system specific, which doesn't make sense for our documentation
Now, generate-docs will compile all subcommands for one command into one document instead of creating a document per subcommand.
For example, instead of creating:
minikbue_addons_enable.md
minikbue_addons_disable.md
it will now create
addons.md
which will include subsections for `minikube addons enable` and `minikube addons disable`.