Skip to content

Commit 7989c02

Browse files
nicohrubecclaude
andcommitted
ci: Skip playwright install-deps for chromium-only jobs
ubuntu-24.04 runners already ship the system libraries that Chromium needs, so the apt-get round-trip from `playwright install-deps` is unnecessary when only chromium is requested. This keeps install-deps for webkit/firefox jobs where the extra system packages are required. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent d8e4d24 commit 7989c02

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

.github/actions/install-playwright/action.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,11 +33,13 @@ runs:
3333
shell: bash
3434
working-directory: ${{ inputs.cwd }}
3535

36+
# ubuntu-24.04 runners already have the system libraries chromium needs (the runner
37+
# ships its own Chromium), so we only run the slow apt-get install-deps for webkit/firefox.
3638
- name: Install Playwright system dependencies only (cached)
3739
env:
3840
PLAYWRIGHT_BROWSERS: ${{ inputs.browsers || 'chromium webkit firefox' }}
3941
run: npx playwright install-deps "$PLAYWRIGHT_BROWSERS"
40-
if: steps.playwright-cache.outputs.cache-hit == 'true'
42+
if: steps.playwright-cache.outputs.cache-hit == 'true' && inputs.browsers != 'chromium'
4143
shell: bash
4244
working-directory: ${{ inputs.cwd }}
4345

0 commit comments

Comments
 (0)