Skip to content

Commit 4e1f12d

Browse files
committed
ci: removes redundant Playwright Chromium download in CI
playwright.config.ts already sets channel: "chrome" on the chromium project, which resolves to the system's branded Google Chrome via Playwright's registry — ubuntu-latest ships Chrome/Chromium/Edge/ Firefox preinstalled. The npx playwright install chromium --with-deps step downloaded a separate Playwright-managed Chromium build that the config never launches, and its apt-based --with-deps step is pure network/CPU overhead once you're on channel: "chrome" (the preinstalled Chrome already carries its own working runtime dependencies). Removing the step entirely eliminates this dead work and the associated network-flake exposure it introduced.
1 parent f9e5c3d commit 4e1f12d

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

‎.github/workflows/ci.yml‎

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,9 @@ jobs:
1818
run: bash scripts/verify-csp-hash.sh
1919
- run: pnpm run typecheck
2020
- run: pnpm test
21-
- name: Install Playwright browsers
22-
run: npx playwright install chromium --with-deps
21+
# No `playwright install` step: playwright.config.ts uses channel: "chrome",
22+
# which resolves to ubuntu-latest's preinstalled Google Chrome — Playwright's
23+
# own bundled Chromium download is never used and would be wasted work.
2324
- name: Run E2E tests
2425
run: pnpm test:e2e
2526
env:

0 commit comments

Comments
 (0)