Skip to content

chore(CI): speed up playwright browser setup#1531

Merged
Timeless0911 merged 1 commit intomainfrom
codex/chore/ci-playwright-install
Mar 13, 2026
Merged

chore(CI): speed up playwright browser setup#1531
Timeless0911 merged 1 commit intomainfrom
codex/chore/ci-playwright-install

Conversation

@Timeless0911
Copy link
Copy Markdown
Contributor

Summary

Use system Chrome for Playwright in CI and drop the explicit browser install step to cut e2e setup time.

Related Links

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

Copilot AI review requested due to automatic review settings March 13, 2026 06:31
@chatgpt-codex-connector
Copy link
Copy Markdown

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

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

Adjusts the Playwright e2e setup to use a preinstalled system Chrome in CI, aiming to reduce CI time by removing the explicit Playwright browser install step.

Changes:

  • Add a shared isCI flag and set Playwright to use the chrome channel in CI.
  • Remove the “Install Playwright browsers” step from the reusable test workflow.

Reviewed changes

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

File Description
tests/playwright.config.ts Uses system Chrome (channel: 'chrome') when CI is set; refactors repeated process.env.CI checks into isCI.
.github/workflows/reusable-test.yml Drops the explicit pnpm playwright install chromium step for e2e jobs.

@Timeless0911 Timeless0911 changed the title ci(playwright): use system chrome in CI chore(CI): speed up playwright browser setup Mar 13, 2026
@Timeless0911 Timeless0911 merged commit 26b09d6 into main Mar 13, 2026
13 checks passed
@Timeless0911 Timeless0911 deleted the codex/chore/ci-playwright-install branch March 13, 2026 07:42
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.

3 participants