Skip to content

Commit

Permalink
not cancelled
Browse files Browse the repository at this point in the history
  • Loading branch information
fjetter committed Jul 30, 2024
1 parent 5b6941f commit 3c1219c
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/ab_tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -108,7 +108,7 @@ jobs:

- name: Upload benchmark results
uses: actions/upload-artifact@v4
if: always()
if: ${{ !cancelled() }}
with:
name: ${{ matrix.runtime-version }}-${{ matrix.repeat }}
path: |
Expand All @@ -119,7 +119,7 @@ jobs:
process-results:
needs: [discover_ab_envs, tests]
name: Combine separate benchmark results
if: always() && fromJson(needs.discover_ab_envs.outputs.matrix).run_AB
if: ${{ !cancelled() && fromJson(needs.discover_ab_envs.outputs.matrix).run_AB }}
runs-on: ubuntu-latest
concurrency:
# Fairly strict concurrency rule to avoid stepping on benchmark db.
Expand Down Expand Up @@ -157,7 +157,7 @@ jobs:
static-site:
needs: [discover_ab_envs, process-results]
# Always generate the site, as this can be skipped even if an indirect dependency fails (like a test run)
if: always() && fromJson(needs.discover_ab_envs.outputs.matrix).run_AB
if: ${{ !cancelled() && fromJson(needs.discover_ab_envs.outputs.matrix).run_AB }}
name: Build static dashboards
runs-on: ubuntu-latest
steps:
Expand Down

0 comments on commit 3c1219c

Please sign in to comment.