Skip to content

Commit

Permalink
chore: reduce number of benchmarking scripts
Browse files Browse the repository at this point in the history
  • Loading branch information
TomAFrench committed Feb 5, 2025
1 parent 410c8b0 commit 668f64c
Showing 1 changed file with 2 additions and 8 deletions.
10 changes: 2 additions & 8 deletions .github/workflows/test-js-packages.yml
Original file line number Diff line number Diff line change
Expand Up @@ -539,7 +539,7 @@ jobs:
TEST_REPORT_NAME=test_report_$NAME
echo "test_report_name=$TEST_REPORT_NAME" >> $GITHUB_OUTPUT
jq --null-input "{ test_reports: [{ name: \"$NAME\", value: (\"$TIME\" | tonumber), unit: \"s\" }]}" > $TEST_REPORT_NAME.json
jq --null-input "{ name: \"$NAME\", value: (\"$TIME\" | tonumber), unit: \"s\" }" > $TEST_REPORT_NAME.json
if [ ! -s $output_file ]; then
# The file is empty so we delete it to signal that `nargo test` failed before it could run any tests
Expand Down Expand Up @@ -606,8 +606,6 @@ jobs:
contents: write

steps:
- uses: actions/checkout@v4

- name: Download matrix test reports
uses: actions/download-artifact@v4
with:
Expand All @@ -616,11 +614,7 @@ jobs:

- name: Merge test reports using jq
run: |
jq --null-input "{ test_reports: [] }" > test_report.json
mv ./.github/scripts/merge-bench-reports.sh merge-bench-reports.sh
./merge-bench-reports.sh test_report
jq ".test_reports" < ./test_report.json > test_bench.json
jq --slurp '.' ./reports/*.json > test_bench.json
- name: Store benchmark result
continue-on-error: true
uses: benchmark-action/github-action-benchmark@4de1bed97a47495fc4c5404952da0499e31f5c29
Expand Down

0 comments on commit 668f64c

Please sign in to comment.