Skip to content

Commit 4b9896a

Browse files
authored
Benchmark fix (#354)
* simplify conditional * don't fetch specific branch * attempt fix * another attempt * reattempt fix benchmark on main * benchmark latest release commit directly * check whether on main * only latest commit
1 parent dfc1a88 commit 4b9896a

File tree

1 file changed

+12
-5
lines changed

1 file changed

+12
-5
lines changed

.github/workflows/test.yml

Lines changed: 12 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -124,17 +124,24 @@ jobs:
124124
PR: 1 # prevents asv from running very compute-intensive benchmarks
125125
PIP_CONSTRAINT: ${{ github.workspace }}/benchmarks/benchmark.txt
126126

127-
- name: Fetch main branch
127+
- name: Fetch latest main
128+
run: git fetch origin main
129+
130+
- name: Fetch latest release commit
131+
run: git fetch origin ${{ fromJSON(steps.latest_release.outputs.data).target_commitish }}
132+
133+
- name: Ensure local main exists
128134
run: |
129-
git fetch origin main
130-
git branch main FETCH_HEAD
135+
CURRENT_BRANCH=$(git rev-parse --abbrev-ref HEAD)
136+
if [ "$CURRENT_BRANCH" != "main" ]; then
137+
git branch main origin/main
138+
fi
131139
132140
- name: Run benchmarks latest release
133-
# here we check if the benchmark on the latest release is not broken
134141
uses: aganders3/headless-gui@v2
135142
with:
136143
run: |
137-
asv run --show-stderr --quick --attribute timeout=300 ${{ fromJSON(steps.latest_release.outputs.data).target_commitish }}^!
144+
asv run --show-stderr --quick --attribute timeout=300 ${{ fromJSON(steps.latest_release.outputs.data).target_commitish }}^!
138145
env:
139146
PR: 1 # prevents asv from running very compute-intensive benchmarks
140147
PIP_CONSTRAINT: ${{ github.workspace }}/benchmarks/benchmark.txt

0 commit comments

Comments
 (0)