[ci] Remove build_devtools_and_process_artifacts as a dependency to run_devtools_e2e_tests (#32267)
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.
lauren committed
Jan 30, 2025 at 11:55 UTC
f02ba2fcc5de5c280d5c79f52302b04daaaf1e23
1 file changed
+12
-6
.github/workflows/runtime_build_and_test.yml
+12
-6
@@ -533,11 +533,6 @@ jobs:
533
RELEASE_CHANNEL: experimental
534
- name: Display structure of build
535
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
536
# Simplifies getting the extension for local testing
537
- name: Archive ${{ matrix.browser }} extension
538
uses: actions/upload-artifact@v4
@@ -545,9 +540,20 @@ jobs:
540
name: react-devtools-${{ matrix.browser }}-extension
541
path: build/devtools/${{ matrix.browser }}-extension.zip
542
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
+
554
run_devtools_e2e_tests:
555
name: Run DevTools e2e tests
550
- needs: build_devtools_and_process_artifacts
556
+ needs: build_and_lint
557
runs-on: ubuntu-latest
558
steps:
559
- uses: actions/checkout@v4