Skip to content

Install Playwright browsers in publish workflow#356

Merged
liuliu-dev merged 1 commit into
mainfrom
liuliu/fix-publish
Jun 9, 2026
Merged

Install Playwright browsers in publish workflow#356
liuliu-dev merged 1 commit into
mainfrom
liuliu/fix-publish

Conversation

@liuliu-dev

Copy link
Copy Markdown
Contributor

The publish workflow fails on npm test because Playwright's Chromium binary isn't available on the runner:

Failed workflow run: https://github.com/github/relative-time-element/actions/runs/27169511742

@liuliu-dev liuliu-dev requested a review from a team as a code owner June 8, 2026 22:04
Copilot AI review requested due to automatic review settings June 8, 2026 22:04
GitHub Advanced Security started work on behalf of liuliu-dev June 8, 2026 22:04 View session
GitHub Advanced Security finished work on behalf of liuliu-dev June 8, 2026 22:04

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 updates the release publish workflow to ensure Playwright’s Chromium browser binary is installed on the GitHub Actions runner before running npm test, aligning it with the existing CI workflow behavior and preventing release-time publish failures.

Changes:

  • Install Playwright Chromium (and required OS deps) prior to running tests in the npm publish job.
  • Install Playwright Chromium (and required OS deps) prior to running tests in the GitHub Packages publish job.
Show a summary per file
File Description
.github/workflows/publish.yml Adds npx playwright install --with-deps chromium before npm test in both publish jobs to prevent missing-browser test failures during releases.

Copilot's findings

Tip

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

  • Files reviewed: 1/1 changed files
  • Comments generated: 0

@liuliu-dev liuliu-dev merged commit 43d5862 into main Jun 9, 2026
6 checks passed
@liuliu-dev liuliu-dev deleted the liuliu/fix-publish branch June 9, 2026 15:28
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