Skip to content

Commit 6a66daa

Browse files
authored
ci: Merge upload actions to fix usage in v4 (#4515)
* ci: Merge upload actions to avoid issues * ci: Ensure bench reports are uploaded w/ unique names * ci: Fix artifact names * ci: Access name for result correctly
1 parent 9d3a4f5 commit 6a66daa

File tree

3 files changed

+10
-15
lines changed

3 files changed

+10
-15
lines changed

.github/workflows/benchmarks.yml

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -30,23 +30,20 @@ jobs:
3030
with:
3131
name: npm-package
3232
- run: mv preact.tgz preact-local.tgz
33-
- name: Upload locally built preact package
34-
uses: actions/upload-artifact@v4
35-
with:
36-
name: bench-environment
37-
path: preact-local.tgz
3833
- name: Download base package
3934
uses: andrewiggins/download-base-artifact@v3
4035
with:
4136
artifact: npm-package
4237
workflow: ci.yml
4338
required: false
4439
- run: mv preact.tgz preact-main.tgz
45-
- name: Upload base preact package
40+
- name: Upload locally build & base preact package
4641
uses: actions/upload-artifact@v4
4742
with:
4843
name: bench-environment
49-
path: preact-main.tgz
44+
path: |
45+
preact-local.tgz
46+
preact-main.tgz
5047
5148
bench_todo:
5249
name: Bench todo

.github/workflows/run-bench.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -107,11 +107,12 @@ jobs:
107107
id: log-artifact-name
108108
run: |
109109
NAME=$(echo "${{ inputs.benchmark }}" | sed -r 's/[\/]+/_/g')
110+
echo "clean_name=$NAME" >> $GITHUB_OUTPUT
110111
echo "artifact_name=logs_$NAME" >> $GITHUB_OUTPUT
111112
- name: Upload results
112113
uses: actions/upload-artifact@v4
113114
with:
114-
name: results
115+
name: results-${{ steps.log-artifact-name.outputs.clean_name }}
115116
path: benchmarks/out/results/${{ inputs.benchmark }}.json
116117
- name: Upload logs
117118
uses: actions/upload-artifact@v4

.github/workflows/single-bench.yml

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -61,11 +61,6 @@ jobs:
6161
with:
6262
name: npm-package
6363
- run: mv preact.tgz preact-local.tgz
64-
- name: Upload locally built preact package
65-
uses: actions/upload-artifact@v4
66-
with:
67-
name: bench-environment
68-
path: preact-local.tgz
6964
- name: Clear working directory
7065
run: |
7166
ls -al
@@ -77,11 +72,13 @@ jobs:
7772
with:
7873
name: base-npm-package
7974
- run: mv preact.tgz preact-main.tgz
80-
- name: Upload base preact package
75+
- name: Upload locally built & base preact package
8176
uses: actions/upload-artifact@v4
8277
with:
8378
name: bench-environment
84-
path: preact-main.tgz
79+
path: |
80+
preact-local.tgz
81+
preact-main.tgz
8582
8683
benchmark:
8784
name: Bench ${{ inputs.benchmark }}

0 commit comments

Comments
 (0)