Skip to content

feat: speed up playwright with prebuilt image#1050

Merged
mbifulco merged 2 commits into
mainfrom
feat/speed-up-playwright
Jul 17, 2025
Merged

feat: speed up playwright with prebuilt image#1050
mbifulco merged 2 commits into
mainfrom
feat/speed-up-playwright

Conversation

@mbifulco

Copy link
Copy Markdown
Owner

No description provided.

@vercel

vercel Bot commented Jul 17, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
mikebifulco-com-bnbu ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 17, 2025 1:38pm

@mbifulco mbifulco requested a review from Copilot July 17, 2025 13:26

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR accelerates end-to-end tests by switching the GitHub Actions runner to a prebuilt Playwright container and simplifying dependency caching.

  • Uses the mcr.microsoft.com/playwright:v1.52.0-noble container with non-root user for tests
  • Replaces manual pnpm store caching with cache: 'pnpm' in actions/setup-node@v4
  • Removes explicit Playwright browser installation step, relying on the container’s prebuilt binaries
Comments suppressed due to low confidence (2)

.github/workflows/test-e2e.yml:37

  • Use pnpm install --frozen-lockfile (or pnpm ci) to ensure the lockfile is strictly honored and prevent unintended dependency updates during CI runs.
        run: pnpm install

.github/workflows/test-e2e.yml:39

  • The playwright install --with-deps step was removed—confirm that mcr.microsoft.com/playwright:v1.52.0-noble includes all required browser binaries and dependencies, or re-add the install step to avoid test failures.
      - name: Run Playwright tests

@mbifulco mbifulco merged commit 72c75c6 into main Jul 17, 2025
8 checks passed
@mbifulco mbifulco deleted the feat/speed-up-playwright branch July 17, 2025 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants