Skip to content

Conversation

@alxndrsn
Copy link
Contributor

@alxndrsn alxndrsn commented Nov 5, 2025

Ref #537 (comment)

This action combines:

  • disabling of man-db updates, and
  • installing playwright browser and its dependencies

I have verified this PR works in these browsers (latest versions):

  • Chrome
  • Firefox
  • Safari (macOS)
  • Safari (iOS)
  • Chrome for Android
  • Not applicable

What else has been done to verify that this works as intended?

CI.

Why is this the best possible solution? Were any other approaches considered?

Clean up mess of #531, as discussed at #537 (comment).

How does this change affect users? Describe intentional changes to behavior and behavior that could have accidentally been affected by code changes. In other words, what are the regression risks?

No effect.

Do we need any specific form for testing your changes? If so, please attach one.

No.

What's changed

Extract separate playwright setup and install steps into a single action.

This action combines:

* disabling of man-db updates, and
* installing playwright browser and its dependencies
@changeset-bot
Copy link

changeset-bot bot commented Nov 5, 2025

⚠️ No Changeset found

Latest commit: 0a26ed5

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@alxndrsn alxndrsn marked this pull request as ready for review November 5, 2025 11:16
@alxndrsn alxndrsn requested a review from garethbowen November 5, 2025 11:17
Copy link
Collaborator

@garethbowen garethbowen left a comment

Choose a reason for hiding this comment

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

Nice one. Very clean.

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