improvements in the make targets for lint checks and automatically fixes (#2379)

- Add target to fix issues automatically
- Add target to check just the ci lint locally

Author: camilamacedo86

Makefile

@@ -69,9 +69,15 @@ tidy: ## Update dependencies
 clean: ## Clean up the build artifacts
 	$(Q)rm -rf build
 
-lint:  ## Run golangci-lint with all checks enabled (development purpose only)
+lint-dev:  ## Run golangci-lint with all checks enabled (development purpose only)
 	./hack/tests/check-lint.sh dev
 
+lint-fix: ## Run golangci-lint automatically fix (development purpose only)
+	./hack/tests/check-lint.sh fix
+
+lint: ## Run golangci-lint with all checks enabled in the ci
+	./hack/tests/check-lint.sh ci
+
 ##############################
 # Generate Artifacts         #
 ##############################
@@ -181,9 +187,8 @@ test: test-unit ## Run the tests
 test-markdown test/markdown:
 	./hack/ci/marker
 
-test-sanity test/sanity: tidy build/operator-sdk
+test-sanity test/sanity: tidy build/operator-sdk lint
 	./hack/tests/sanity-check.sh
-	./hack/tests/check-lint.sh ci
 
 TEST_PKGS:=$(shell go list ./... | grep -v -P 'github.com/operator-framework/operator-sdk/(hack|test/e2e)')
 test-unit test/unit: ## Run the unit tests

hack/tests/check-lint.sh

@@ -19,8 +19,17 @@ DEV_LINTERS=(
     "--enable=gosec"  # NOT add this one to CI since was defined that it should be optional for now at least.
 )
 
+# Some lint checks can be fixed automatically by using it.
+FIX_LINTERS=(
+    "--fix"
+)
+
 subcommand=$1
 case $subcommand in
+	"fix")
+	  header_text "Running lint check with automatically fix"
+		LINTERS=${FIX_LINTERS[@]}
+		;;
 	"dev")
 	  ##todo(camilamacedo86): It should be removed when all linter checks be enabled
 	  header_text "Checking the project with all linters (dev)"