[ci] let all workflow use new implementation

Signed-off-by: Avimitin <dev@avimit.in>
chipsalliance · Jul 29, 2023 · cfa93ce · cfa93ce
1 parent c875cab
commit cfa93ce
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/.github/workflows/gen-test-elf.yml b/.github/workflows/gen-test-elf.yml
@@ -30,7 +30,7 @@ jobs:
           install_url: https://releases.nixos.org/nix/nix-2.13.3/install
           nix_path: nixpkgs=channel:nixos-unstable
       - id: ci-tests
-        run: nix-shell -p ammonite --run 'make gen-test-case-bucket'
+        run: nix develop -c make gen-test-case-bucket
 
   gen-test-artifacts:
     needs: [gen-matrix]

diff --git a/.github/workflows/pr.yml b/.github/workflows/pr.yml
@@ -57,12 +57,11 @@ jobs:
         run: nix develop .#testcase -c make ci-run NAME=${{ matrix.name }}
 
       - uses: actions/upload-artifact@v3
-        if: always()
+        if: failure()
         with:
-          name: results
+          name: fail-logs
           path: |
-            result-*.md
-            perf-result-*.md
+            ./test-log/fail
 
       - uses: actions/upload-artifact@v3
         if: always()