Skip to content

Commit

Permalink
Don't run the Krausest benchmark unless we have to
Browse files Browse the repository at this point in the history
  • Loading branch information
NullVoxPopuli committed Sep 11, 2024
1 parent ae5b971 commit 5f70506
Showing 1 changed file with 16 additions and 4 deletions.
20 changes: 16 additions & 4 deletions .github/workflows/perf.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,11 @@ name: PerformanceCheck

on:
workflow_dispatch:
push:
branches:
- main
# Don't run on every merge to main, because many merges
# may not even be vm related, but infra, or GH Actions
# push:
# branches:
# - main
pull_request:
branches: [main]

Expand All @@ -28,24 +30,34 @@ jobs:
- uses: actions/checkout@v4
with:
fetch-depth: 0

- uses: marceloprado/[email protected]
id: did-change
with:
paths: packages/

- uses: wyvox/action-setup-pnpm@v3
if: steps.did-change.outputs.changed == 'true'
with:
node-version: '20.1.0'

- name: RUN
if: steps.did-change.outputs.changed == 'true'
run: pnpm run benchmark:setup

- name: Remove unused artifacts
if: steps.did-change.outputs.changed == 'true'
run: rm -rf ./tracerbench-results/traces && rm -rf ./tracerbench-results/traces.zip

- name: Upload Tracerbench Artifacts
if: failure() || success()
if: steps.did-change.outputs.changed == 'true' && (failure() || success())
uses: actions/upload-artifact@v3
with:
name: Trace Artifacts
path: tracerbench-results

- name: Write message
if: steps.did-change.outputs.changed == 'true'
uses: mshick/add-pr-comment@v2
with:
message-path: "tracerbench-results/msg.txt"
Expand Down

0 comments on commit 5f70506

Please sign in to comment.