Skip to content

Commit

Permalink
Minor refinements + better consistency
Browse files Browse the repository at this point in the history
  • Loading branch information
MahdiBM committed Jan 10, 2025
1 parent d9a9700 commit c743d5c
Showing 1 changed file with 6 additions and 9 deletions.
15 changes: 6 additions & 9 deletions .github/workflows/benchmark.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ on:
branches: [main]
types: [synchronize]
paths:
- "Sources/*.swift"
- Sources/*.swift
- .github/workflows/benchmark.yml

jobs:
Expand Down Expand Up @@ -44,6 +44,7 @@ jobs:
- name: Install jemalloc, curl, jq and zstd
run: |
set -eu
apt-get update -y
apt-get install -y libjemalloc-dev curl jq zstd
Expand All @@ -62,10 +63,11 @@ jobs:
benchmark baseline update \
'${{ github.head_ref || github.ref_name }}'
- name: Read Benchmark Result
- name: Read benchmark result
id: read-benchmark
run: |
set -eu
swift package -c release --disable-sandbox \
--package-path Benchmarks \
benchmark baseline read \
Expand Down Expand Up @@ -167,10 +169,7 @@ jobs:
echo 'EOF' >> "${GITHUB_ENV}"
- name: Output the comment as job summary
run: |
set -eu
echo '${{ env.PR_COMMENT }}' >> "${GITHUB_STEP_SUMMARY}"
run: echo '${{ env.PR_COMMENT }}' >> "${GITHUB_STEP_SUMMARY}"

# There is a '<!-- benchmark ci tag. exit code: {some_number} -->' comment at the beginning of the benchamrk report comment.
# The number in that comment is the exit code of the last benchmark run.
Expand Down Expand Up @@ -253,10 +252,8 @@ jobs:
-d "$BODY_JSON" \
"$ENDPOINT"
- name: Exit With Correct Status
- name: Exit with correct status
run: |
set -eu
EXIT_CODE='${{ steps.compare-benchmark.outputs.exit-status }}'
echo "Previous exit code was: $EXIT_CODE"
exit $EXIT_CODE

0 comments on commit c743d5c

Please sign in to comment.