Merge pull request #576 from Vampire/use-if-always

dorny · May 19, 2025 · 0b4ea9b · 0b4ea9b
2 parents 890a17c + 302102c
commit 0b4ea9b
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -24,7 +24,7 @@ jobs:
       - run: npm test
 
       - name: Upload test results
-        if: success() || failure()
+        if: ${{ !cancelled() }}
         uses: actions/upload-artifact@v4
         with:
           name: test-results

diff --git a/.github/workflows/manual-run.yml b/.github/workflows/manual-run.yml
@@ -15,7 +15,7 @@ jobs:
 
       - name: Create test report
         uses: ./
-        if: success() || failure()
+        if: ${{ !cancelled() }}
         with:
           name: JEST Tests
           path: __tests__/__results__/*.xml

diff --git a/README.md b/README.md
@@ -50,7 +50,7 @@ jobs:
 
       - name: Test Report
         uses: dorny/test-reporter@v2
-        if: success() || failure()    # run this step even if previous step failed
+        if: ${{ !cancelled() }}       # run this step even if previous step failed
         with:
           name: JEST Tests            # Name of the check run which will be created
           path: reports/jest-*.xml    # Path to test results
@@ -79,7 +79,7 @@ jobs:
       - run: npm ci                       # install packages
       - run: npm test                     # run tests (configured to use jest-junit reporter)
       - uses: actions/upload-artifact@v4  # upload test results
-        if: success() || failure()        # run this step even if previous step failed
+        if: ${{ !cancelled() }}           # run this step even if previous step failed
         with:
           name: test-results
           path: jest-junit.xml