Merge pull request #9 from slintes/prepare_openshift_ci

Cleanup and prepare Openshift CI

.ci-operator.yaml

@@ -0,0 +1,4 @@
+build_root_image:
+  namespace: openshift
+  name: release
+  tag: rhel-8-release-golang-1.20-openshift-4.14

.github/workflows/pre-submit.yaml

@@ -19,11 +19,11 @@ jobs: # jobs to run
       with:
         fetch-depth: 0
 
-    - name: Verify imports are sorted
-      run: make test-imports
+    - name: Build
+      run: make build
 
-    - name: Verify there are no packages to vendor
-      run: make verify-vendor
+    - name: Run all linters etc.
+      run: make lint
 
-    - name: Run checks and unit tests
-      run: make check
+    - name: Run unit tests
+      run: make test

Makefile

@@ -29,17 +29,24 @@ rm -rf $$TMP_DIR ;\
 }
 endef
 
+.PHONY: build
+build: ## Build.
+	go build ./...
+
 .PHONY: test
-test: ## Run tests.
+test: ## Run unit tests.
 	go test ./... -coverprofile cover.out -v
 
+.PHONY: lint
+lint: tidy goimports fix-imports vet verify-no-changes ## Run linters etc.
+
 .PHONY: check
-check: fmt vet goimports verify-no-changes ## Dockerized version of make test with additional verifications
-	$(DOCKER_GO) "make test"
+check: ## Dockerized version of make test with additional verifications
+	$(DOCKER_GO) "make lint test"
 
-.PHONY: fmt
-fmt: ## Run go fmt against code.
-	go fmt ./...
+.PHONY: goimports
+goimports: install-goimports ## Run go fmt against code.
+	$(GOIMPORTS) -w ./pkg
 
 .PHONY: vet
 vet: ## Run go vet against code.
@@ -49,17 +56,10 @@ vet: ## Run go vet against code.
 verify-no-changes: ## verify there are no un-staged changes
 	./hack/verify-diff.sh
 
-.PHONY:vendor
-vendor: ## Runs go mod vendor
-	go mod vendor
-
 .PHONY: tidy
 tidy: ## Runs go mod tidy
 	go mod tidy
 
-.PHONY:verify-vendor
-verify-vendor:tidy vendor verify-no-changes ##Verifies vendor and tidy didn't cause changes
-
 SORT_IMPORTS = $(shell pwd)/bin/sort-imports
 .PHONY: sort-imports
 sort-imports: ## Download sort-imports locally if necessary.
@@ -74,6 +74,6 @@ fix-imports: sort-imports ## Sort imports
 	$(SORT_IMPORTS) . -w
 
 GOIMPORTS = $(shell pwd)/bin/goimports
-goimports: ## updates goimports.
+.PHONY: install-goimports
+install-goimports: ## updates goimports.
 	$(call go-install-tool,$(GOIMPORTS),golang.org/x/tools/cmd/goimports@$(GOIMPORTS_VERSION))
-	$(GOIMPORTS) -w ./pkg