Thanks to visit codestin.com
Credit goes to github.com

Skip to content

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.

@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