Skip to content

ci: shard over browser, cache.#44

Merged
knightedcodemonkey merged 2 commits intomainfrom
bananas
Mar 29, 2026
Merged

ci: shard over browser, cache.#44
knightedcodemonkey merged 2 commits intomainfrom
bananas

Conversation

@knightedcodemonkey
Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings March 29, 2026 14:12
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the Playwright CI setup to run E2E tests per browser (via a matrix) and reduce repeat work across runs through caching, aligning CI execution more closely with Playwright’s multi-project model.

Changes:

  • Run Playwright workflow as a browser matrix (chromium, webkit) and pass --project to target each run.
  • Add npm dependency caching and a dedicated cache for Playwright browser downloads.
  • Limit Playwright workers on CI to a fixed value.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
playwright.config.ts Caps CI worker parallelism to 2 for more predictable execution.
.github/workflows/playwright.yml Splits E2E runs by browser using a matrix and adds caching for npm + Playwright browsers.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@knightedcodemonkey knightedcodemonkey merged commit 30735a6 into main Mar 29, 2026
1 check passed
@knightedcodemonkey knightedcodemonkey deleted the bananas branch March 29, 2026 14:17
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