refactor: move shellcheck under linters (#133)

Signed-off-by: Artur Troian <[email protected]>

.github/workflows/shellcheck.yaml → .github/workflows/lint-shell.yaml

@@ -1,13 +1,12 @@
-name: shellcheck
-
+---
+name: lint
 on:
   push:
     paths:
       - "**.sh"
-      - ".github/workflows/shellcheck.yaml"
-
+      - ".github/workflows/lint-shell.yaml"
 jobs:
-  shellcheck-lint:
+  shell:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
@@ -17,4 +16,4 @@ jobs:
       - uses: actions/setup-go@v3
         with:
           go-version: "${{ env.GOLANG_VERSION }}"
-      - run: make shellcheck
+      - run: make lint-shell

make/lint.mk

@@ -1,3 +1,7 @@
+LINT ?= go \
+proto \
+shell
+
 BUF_LINT_PACKAGES ?= provider \
 node
 
@@ -16,8 +20,16 @@ lint-proto-%:
 .PHONY: lint-proto
 lint-proto: $(BUF) $(patsubst %, lint-proto-%,$(BUF_LINT_PACKAGES))
 
+.PHONY: lint-shell
+lint-shell:
+	docker run --rm \
+	--volume $(PWD):/shellcheck \
+	--entrypoint sh \
+	koalaman/shellcheck-alpine:stable \
+	-x /shellcheck/script/shellcheck.sh
+
 .PHONY: lint
-lint: lint-go lint-proto
+lint: $(patsubst %, lint-%,$(LINT))
 
 .PHONY: check-breaking
 proto-check-breaking: $(BUF)

make/test.mk

@@ -2,14 +2,6 @@ TEST_MODULE   ?= ./...
 
 COVER_PACKAGES = $(shell go list ./... | grep -v mock | paste -sd, -)
 
-.PHONY: shellcheck
-shellcheck:
-	docker run --rm \
-	--volume $(PWD):/shellcheck \
-	--entrypoint sh \
-	koalaman/shellcheck-alpine:stable \
-	-x /shellcheck/script/shellcheck.sh
-
 TEST_TIMEOUT  ?= 300
 TEST_RACE     ?= 0
 TEST_NOCACHE  ?= 0