Makefile: add a target to run the auto-fixer (Azure#4247)

* Makefile: correctly depend on tools

The entire point of having per-tool Makefile targets is so that each
top-level target that requires some tool can ensure that the tools it
uses are installed as necessary. Having a top-level `install-tools`
target is fine, if you need to do it, but using it exclusively loses the
whole point of incremental installs and revalidations from Make.

Signed-off-by: Steve Kuznetsov <stekuznetsov@microsoft.com>

* Makefile: add a target to run the auto-fixer

The --fix argument to the linter will automatically update the AST where
possible to make the codebase conformant.

Signed-off-by: Steve Kuznetsov <stekuznetsov@microsoft.com>

---------

Signed-off-by: Steve Kuznetsov <stekuznetsov@microsoft.com>

Author: stevekuznetsov

Makefile

@@ -283,7 +283,7 @@ validate-go: validate-imports
 	go test -tags e2e -run ^$$ ./test/e2e/...
 
 .PHONY: validate-go-action
-validate-go-action: validate-imports
+validate-go-action: validate-imports validate-lint-go-fix
 	go run ./hack/licenses -validate -ignored-go vendor,pkg/client,.git -ignored-python python/client,python/az/aro/azext_aro/aaz,vendor,.git
 	@[ -z "$$(ls pkg/util/*.go 2>/dev/null)" ] || (echo error: go files are not allowed in pkg/util, use a subpackage; exit 1)
 	@[ -z "$$(find -name "*:*")" ] || (echo error: filenames with colons are not allowed on Windows, please rename; exit 1)
@@ -307,6 +307,18 @@ unit-test-go-coverpkg: $(GOTESTSUM)
 lint-go: $(GOLANGCI_LINT)
 	$(GOLANGCI_LINT) run --verbose
 
+.PHONY: lint-go-fix
+lint-go-fix: $(GOLANGCI_LINT)
+	$(GOLANGCI_LINT) run --verbose --fix
+
+.PHONY: validate-lint-go-fix
+validate-lint-go-fix: lint-go-fix
+	if ! git diff --quiet HEAD; then \
+		git diff; \
+		echo "You need to run 'make lint-go-fix' to update the codebase and commit the changes"; \
+		exit 1; \
+	fi
+
 .PHONY: lint-admin-portal
 lint-admin-portal:
 	docker build --platform=linux/amd64 --build-arg REGISTRY=$(REGISTRY) --build-arg BUILDER_REGISTRY=$(BUILDER_REGISTRY) -f Dockerfile.portal_lint . -t linter:latest --no-cache