use separate workflow for linters

Author: okdas

.github/workflows/pr.yml .github/workflows/ggshield.yml

@@ -1,6 +1,8 @@
 name: GitGuardian scan
 
-on: [push, pull_request]
+on:
+  workflow_dispatch:
+  push:
 
 jobs:
   scanning:
@@ -16,6 +18,6 @@ jobs:
         env:
           GITHUB_PUSH_BEFORE_SHA: ${{ github.event.before }}
           GITHUB_PUSH_BASE_SHA: ${{ github.event.base }}
-          GITHUB_PULL_BASE_SHA:  ${{ github.event.pull_request.base.sha }}
+          GITHUB_PULL_BASE_SHA: ${{ github.event.pull_request.base.sha }}
           GITHUB_DEFAULT_BRANCH: ${{ github.event.repository.default_branch }}
           GITGUARDIAN_API_KEY: ${{ secrets.GITGUARDIAN_API_KEY }}

.github/workflows/linters.yml

@@ -0,0 +1,26 @@
+name: Linters, static checks and code coverage
+
+on:
+  workflow_dispatch:
+  # push:
+  #   branches: [main, dk-add-linters]
+  #   paths-ignore:
+  #     - "docs/**"
+  #     - "**.md"
+  # pull_request:
+  #   paths-ignore:
+  #     - "docs/**"
+  #     - "**.md"
+
+jobs:
+  run-linters:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v3
+      # - name: Run static check
+      #   uses: dominikh/[email protected]
+      # - name: Run static check
+      #   uses: reviewdog/action-staticcheck@v1
+      # with:
+      #   # TODO: pick filter https://github.com/reviewdog/reviewdog#filter-mode
+      #   filter_mode: nofilter

.github/workflows/main.yml

@@ -62,12 +62,19 @@ jobs:
         # Only run if the test failed on target version to avoid duplicated annotations on GitHub.
         if: ${{ always() && env.TARGET_GOLANG_VERSION == matrix.go }}
         uses: reviewdog/action-staticcheck@v1
+        with:
+          # TODO: pick filter https://github.com/reviewdog/reviewdog#filter-mode
+          filter_mode: nofilter
+      - name: Run golangci-lint
+        # Only run if the test failed on target version to avoid duplicated annotations on GitHub.
+        if: ${{ always() && env.TARGET_GOLANG_VERSION == matrix.go }}
+        uses: golangci/golangci-lint-action@v3
       # TODO(@okdas): move coverage to a separate job
       - name: create coverage report
-        if: ${{ env.TARGET_GOLANG_VERSION == matrix.go }}
+        if: ${{ always() && env.TARGET_GOLANG_VERSION == matrix.go }}
         run: make test_all_with_coverage
       - name: Upload coverage to Codecov
-        if: ${{ env.TARGET_GOLANG_VERSION == matrix.go }}
+        if: ${{ always() && env.TARGET_GOLANG_VERSION == matrix.go }}
         uses: codecov/codecov-action@v3
 
   # TODO(@okdas): reuse artifacts built by the previous job instead