Skip to content

Commit 791e87b

Browse files
committed
[ci] Remove build_devtools_and_process_artifacts as a dependency to run_devtools_e2e_tests
I just noticed that we don't actually need to let the devtools build finish first because the e2e tests don't use those built files. We can decouple them to allow them to run in paralllel.
1 parent 14f7c07 commit 791e87b

File tree

1 file changed

+12
-6
lines changed

1 file changed

+12
-6
lines changed

.github/workflows/runtime_build_and_test.yml

Lines changed: 12 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -533,21 +533,27 @@ jobs:
533533
RELEASE_CHANNEL: experimental
534534
- name: Display structure of build
535535
run: ls -R build
536-
- name: Archive devtools build
537-
uses: actions/upload-artifact@v4
538-
with:
539-
name: react-devtools-${{ matrix.browser }}
540-
path: build/devtools.tgz
541536
# Simplifies getting the extension for local testing
542537
- name: Archive ${{ matrix.browser }} extension
543538
uses: actions/upload-artifact@v4
544539
with:
545540
name: react-devtools-${{ matrix.browser }}-extension
546541
path: build/devtools/${{ matrix.browser }}-extension.zip
547542

543+
merge_devtools_artifacts:
544+
name: Merge DevTools artifacts
545+
needs: build_devtools_and_process_artifacts
546+
runs-on: ubuntu-latest
547+
steps:
548+
- name: Merge artifacts
549+
uses: actions/upload-artifact/merge@v4
550+
with:
551+
name: react-devtools
552+
pattern: react-devtools-*-extension
553+
548554
run_devtools_e2e_tests:
549555
name: Run DevTools e2e tests
550-
needs: build_devtools_and_process_artifacts
556+
needs: build_and_lint
551557
runs-on: ubuntu-latest
552558
steps:
553559
- uses: actions/checkout@v4

0 commit comments

Comments
 (0)