fix: remove vendor folder from semgrep scan and fix workflow (#3959)

* chore: remove vendor folder from semgrep scan Signed-off-by: Jorge Turrado <[email protected]> * fix the checkout Signed-off-by: Jorge Turrado <[email protected]> * add ne wline Signed-off-by: Jorge Turrado <[email protected]> Signed-off-by: Jorge Turrado <[email protected]>
kedacore · Dec 6, 2022 · ece1d19 · ece1d19
1 parent 8f9b1d1
commit ece1d19
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/.github/workflows/static-analysis.yml b/.github/workflows/static-analysis.yml
@@ -20,14 +20,14 @@ jobs:
       uses: actions/checkout@v3
 
     - name: Register workspace path
-      if: github.ref != 'refs/heads/main'
+      if: ${{ github.event.number > 0 }}
       run: git config --global --add safe.directory "$GITHUB_WORKSPACE"
 
     - name: Checkout Pull Request
       env:
         GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
       id: checkout
-      if: github.ref != 'refs/heads/main'
+      if: ${{ github.event.number > 0 }}
       run: |
         gh pr checkout ${{ github.event.number }}
 
@@ -60,14 +60,14 @@ jobs:
     steps:
       - uses: actions/checkout@v3
       - name: Register workspace path
-        if: github.ref != 'refs/heads/main'
+        if: ${{ github.event.number > 0 }}
         run: git config --global --add safe.directory "$GITHUB_WORKSPACE"
 
       - name: Checkout Pull Request
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
         id: checkout
-        if: github.ref != 'refs/heads/main'
+        if: ${{ github.event.number > 0 }}
         run: |
           gh pr checkout ${{ github.event.number }}
 

diff --git a/.semgrepignore b/.semgrepignore
@@ -0,0 +1,2 @@
+# Common large paths
+vendor/