actions: Remove lint (#1040)

We have found weird issues with the actions and the feature it has of showing the line at the code review is not used much. This change move the lint to "check" target so it's run as part of prow unit-test. Signed-off-by: Enrique Llorente <[email protected]>
nmstate · Apr 7, 2022 · 3699213 · 3699213
1 parent 6c8c3f1
commit 3699213
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 26 deletions.
diff --git a/.github/workflows/check.yaml b/.github/workflows/check.yaml
diff --git a/Makefile b/Makefile
@@ -92,7 +92,7 @@ SKIP_IMAGE_BUILD ?= false
 
 all: check handler
 
-check: vet whitespace-check gofmt-check
+check: lint vet whitespace-check gofmt-check
 
 format: whitespace-format gofmt
 

diff --git a/hack/lint.sh b/hack/lint.sh
@@ -3,8 +3,8 @@ golangci_lint_version=v1.42.1
 if [ ! -f $(go env GOPATH)/bin/golangci-lint ]; then
     curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b $(go env GOPATH)/bin $golangci_lint_version
 fi
-golangci-lint run
+golangci-lint run --timeout 5m0s
 (
 	cd api
-	golangci-lint run --config ../.golangci.yml
-)
+	golangci-lint run --timeout 5m0s --config ../.golangci.yml
+)