Skip to content

Commit 1ef327d

Browse files
paddymulclaude
andcommitted
ci: remove redundant pnpm setup from Playwright test jobs
The test scripts (test_playwright_marimo.sh, test_playwright_wasm_marimo.sh, test_playwright_jupyter.sh) already run their own `pnpm install` / `npm install` in packages/buckaroo-js-core/. The CI-level pnpm setup + workspace-wide `pnpm install --frozen-lockfile` was redundant. Node.js is still set up since Playwright requires it. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 8897a64 commit 1ef327d

1 file changed

Lines changed: 3 additions & 33 deletions

File tree

.github/workflows/checks.yml

Lines changed: 3 additions & 33 deletions
Original file line numberDiff line numberDiff line change
@@ -250,18 +250,8 @@ jobs:
250250
enable-cache: true
251251
python-version: "3.13"
252252
prune-cache: false
253-
- name: Setup pnpm
254-
uses: pnpm/action-setup@v4
255-
with:
256-
version: 9.10.0
257-
- name: Setup Node.js with pnpm cache
253+
- name: Setup Node.js
258254
uses: actions/setup-node@v4
259-
with:
260-
cache: 'pnpm'
261-
cache-dependency-path: 'packages/pnpm-lock.yaml'
262-
- name: Install pnpm dependencies
263-
working-directory: packages
264-
run: pnpm install --frozen-lockfile
265255
- name: Download build artifacts
266256
uses: actions/download-artifact@v4
267257
with:
@@ -290,18 +280,8 @@ jobs:
290280
enable-cache: true
291281
python-version: "3.13"
292282
prune-cache: false
293-
- name: Setup pnpm
294-
uses: pnpm/action-setup@v4
295-
with:
296-
version: 9.10.0
297-
- name: Setup Node.js with pnpm cache
283+
- name: Setup Node.js
298284
uses: actions/setup-node@v4
299-
with:
300-
cache: 'pnpm'
301-
cache-dependency-path: 'packages/pnpm-lock.yaml'
302-
- name: Install pnpm dependencies
303-
working-directory: packages
304-
run: pnpm install --frozen-lockfile
305285
- name: Download build artifacts
306286
uses: actions/download-artifact@v4
307287
with:
@@ -364,18 +344,8 @@ jobs:
364344
enable-cache: true
365345
python-version: "3.13"
366346
prune-cache: false
367-
- name: Setup pnpm
368-
uses: pnpm/action-setup@v4
369-
with:
370-
version: 9.10.0
371-
- name: Setup Node.js with pnpm cache
347+
- name: Setup Node.js
372348
uses: actions/setup-node@v4
373-
with:
374-
cache: 'pnpm'
375-
cache-dependency-path: 'packages/pnpm-lock.yaml'
376-
- name: Install pnpm dependencies
377-
working-directory: packages
378-
run: pnpm install --frozen-lockfile
379349
- name: Download build artifacts
380350
uses: actions/download-artifact@v4
381351
with:

0 commit comments

Comments
 (0)