Skip to content

Commit 435dbed

Browse files
eakmanrqclaude
andcommitted
Fix CI: revert ui-test to Playwright container approach
Using ubuntu-latest with --with-deps downloads hundreds of system packages. Revert to using the Playwright container which has all browser dependencies pre-installed. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: eakmanrq <6326532+eakmanrq@users.noreply.github.com>
1 parent 2126fc8 commit 435dbed

File tree

1 file changed

+8
-16
lines changed

1 file changed

+8
-16
lines changed

.github/workflows/pr.yaml

Lines changed: 8 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -219,26 +219,18 @@ jobs:
219219
needs.changes.outputs.client == 'true' || needs.changes.outputs.ci ==
220220
'true' || github.ref == 'refs/heads/main'
221221
runs-on: ubuntu-latest
222+
container:
223+
image: mcr.microsoft.com/playwright:v1.54.1-jammy
222224
steps:
223225
- uses: actions/checkout@v5
224-
- uses: actions/setup-node@v6
225-
with:
226-
node-version: '20'
227-
- uses: pnpm/action-setup@v4
228-
with:
229-
version: latest
230-
- name: Get pnpm store directory
231-
id: pnpm-cache
232-
run: echo "store=$(pnpm store path)" >> $GITHUB_OUTPUT
233-
- uses: actions/cache@v4
234-
with:
235-
path: ${{ steps.pnpm-cache.outputs.store }}
236-
key: pnpm-store-${{ hashFiles('pnpm-lock.yaml') }}
237-
restore-keys: pnpm-store-
226+
- name: Install pnpm via corepack
227+
run: |
228+
npm install --global corepack@latest
229+
corepack enable
230+
corepack prepare pnpm@latest-10 --activate
231+
pnpm config set store-dir .pnpm-store
238232
- name: Install dependencies
239233
run: pnpm install
240-
- name: Install Playwright browsers
241-
run: pnpm --prefix web/client exec playwright install --with-deps
242234
- name: Run tests
243235
run: npm --prefix web/client run test
244236

0 commit comments

Comments
 (0)