actions: Remove lint (#1044)

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]> Co-authored-by: Enrique Llorente <[email protected]>
nmstate · Apr 7, 2022 · 9a7bf91 · 9a7bf91
1 parent de84200
commit 9a7bf91
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
@@ -93,7 +93,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
+)