Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade Playwright to the latest version #940

Merged
merged 1 commit into from
Mar 14, 2025

Conversation

jonkoops
Copy link
Collaborator

Upgrades Playwright to the latest version, this allows it to install browsers on newer distributions such as Ubuntu 24.04 LTS, which is used on GitHub Actions.

Signed-off-by: Jon Koops <jonkoops@gmail.com>
@jonkoops jonkoops force-pushed the upgrade-playwright branch from 6f4dc64 to d599e1c Compare March 12, 2025 17:29
@jonkoops
Copy link
Collaborator Author

jonkoops commented Mar 12, 2025

Note that the CI will fail on the Node.js tests, this is not due to the changes in this PR, but due to the fact that Node.js introduced support for Blob as a global. There seems to be a bug in the JSZip that wasn't previously caught here. See #941.

@jonkoops
Copy link
Collaborator Author

@Stuk could I ask you for a review of this one?

Copy link
Owner

@Stuk Stuk left a comment

Choose a reason for hiding this comment

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

Thanks! What was the reason to delete package-lock.json though?

@jonkoops
Copy link
Collaborator Author

Thanks! What was the reason to delete package-lock.json though?

I haven't deleted it, but the diff is quite big because of the change in transitive dependencies, making it look like it was removed. If you click 'load diff' in the GitHub UI it should show you the actual changes.

@Stuk
Copy link
Owner

Stuk commented Mar 14, 2025

Ah, my mistake!

@jonkoops
Copy link
Collaborator Author

Merging this in, even though CI is failing as this is unrelated to the changes of the PR. Either way, we'll fix CI later.

@jonkoops jonkoops merged commit 418c11f into Stuk:main Mar 14, 2025
1 check failed
@jonkoops jonkoops deleted the upgrade-playwright branch March 14, 2025 16:52
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