Skip to content

Commit

Permalink
[ci] Remove dep on build_devtools from run_devtools_e2e_tests
Browse files Browse the repository at this point in the history
The run_devtools_e2e_tests job does not appear to actually need anything
from the prior job since it always builds inline. Removing this dep
allows us to run both in parallel.

ghstack-source-id: 62e9eeb7ea7af0cbf94934669bf5510b28f460fe
Pull Request resolved: #30348
  • Loading branch information
poteto committed Jul 16, 2024
1 parent 4da4cb8 commit 2bd7c7a
Showing 1 changed file with 2 additions and 6 deletions.
8 changes: 2 additions & 6 deletions .github/workflows/runtime_build_and_test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -308,6 +308,7 @@ jobs:
uses: actions/download-artifact@v4
with:
path: build
pattern: build_*
merge-multiple: true
- name: Display structure of build
run: ls -R build
Expand Down Expand Up @@ -336,6 +337,7 @@ jobs:
uses: actions/download-artifact@v4
with:
path: build
pattern: build_*
merge-multiple: true
- run: ./scripts/circleci/pack_and_store_devtools_artifacts.sh
env:
Expand All @@ -361,7 +363,6 @@ jobs:

run_devtools_e2e_tests:
name: Run DevTools e2e tests
needs: build_devtools_and_process_artifacts
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
Expand All @@ -377,11 +378,6 @@ jobs:
path: "**/node_modules"
key: ${{ runner.arch }}-${{ runner.os }}-modules-${{ hashFiles('yarn.lock') }}
- run: yarn install --frozen-lockfile
- name: Restore archived build
uses: actions/download-artifact@v4
with:
path: build
merge-multiple: true
- run: |
npx playwright install
sudo npx playwright install-deps
Expand Down

0 comments on commit 2bd7c7a

Please sign in to comment.