Run PR checks using JS only

github · Sep 24, 2024 · 6b2f7e7 · 6b2f7e7
1 parent af8e2bc
commit 6b2f7e7
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 28 deletions.
diff --git a/.github/workflows/__zstd-bundle-fallback.yml b/.github/workflows/__zstd-bundle-fallback.yml
diff --git a/.github/workflows/__zstd-bundle.yml b/.github/workflows/__zstd-bundle.yml
diff --git a/pr-checks/checks/zstd-bundle-fallback.yml b/pr-checks/checks/zstd-bundle-fallback.yml
@@ -21,12 +21,8 @@ steps:
   - id: init
     uses: ./../action/init
     with:
-      # Swift is not supported on Ubuntu so we manually exclude it from the list here
-      languages: cpp,csharp,go,java,javascript,python,ruby 
+      languages: javascript
       tools: ${{ steps.prepare-test.outputs.tools-url }}
-  - name: Build code
-    shell: bash
-    run: ./build.sh
   - uses: ./../action/analyze
     with:
       output: ${{ runner.temp }}/results
@@ -35,12 +31,12 @@ steps:
     uses: actions/upload-artifact@v3
     with:
       name: zstd-bundle.sarif
-      path: ${{ runner.temp }}/results/cpp.sarif
+      path: ${{ runner.temp }}/results/javascript.sarif
       retention-days: 7
   - name: Check expected diagnostics
     uses: actions/github-script@v7
     env:
-      SARIF_PATH: ${{ runner.temp }}/results/cpp.sarif
+      SARIF_PATH: ${{ runner.temp }}/results/javascript.sarif
     with:
       script: |
         const fs = require('fs');

diff --git a/pr-checks/checks/zstd-bundle.yml b/pr-checks/checks/zstd-bundle.yml
@@ -20,12 +20,8 @@ steps:
   - id: init
     uses: ./../action/init
     with:
-      # Swift is not supported on Ubuntu so we manually exclude it from the list here
-      languages: cpp,csharp,go,java,javascript,python,ruby 
+      languages: javascript
       tools: ${{ steps.prepare-test.outputs.tools-url }}
-  - name: Build code
-    shell: bash
-    run: ./build.sh
   - uses: ./../action/analyze
     with:
       output: ${{ runner.temp }}/results
@@ -34,12 +30,12 @@ steps:
     uses: actions/upload-artifact@v3
     with:
       name: zstd-bundle.sarif
-      path: ${{ runner.temp }}/results/cpp.sarif
+      path: ${{ runner.temp }}/results/javascript.sarif
       retention-days: 7
   - name: Check diagnostic with expected tools URL appears in SARIF
     uses: actions/github-script@v7
     env:
-      SARIF_PATH: ${{ runner.temp }}/results/cpp.sarif
+      SARIF_PATH: ${{ runner.temp }}/results/javascript.sarif
     with:
       script: |
         const fs = require('fs');