-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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 1.30.0 #48007
Upgrade Playwright to 1.30.0 #48007
Conversation
Size Change: +1.05 kB (0%) Total Size: 1.33 MB
ℹ️ View Unchanged
|
Flaky tests detected in 32b51c9. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/4173876911
|
Co-authored-by: Glen Davies <glendaviesnz@users.noreply.github.com>
This one is ready for review now :). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
v1.30 yay! 🥳
I've left just a couple nit
comments. Nice work, thank you! 🙌
Co-authored-by: Bart Kalisz <bartlomiej.kalisz@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🚢
Special thanks for keeping on top of the best practices documentation 🙇
Thank you, @kevin940726! |
I just cherry-picked this PR to the wp/6.2 branch to get it included in the next release: be68ed7 |
* Upgrade Playwright to 1.30.0 * Update doc for best practices * Fix snapshot path * Fix strict locator * Try to fix safari test * Update image test to use visual testing * Update docs/contributors/code/e2e/README.md Co-authored-by: Glen Davies <glendaviesnz@users.noreply.github.com> * Drop unnecessary await in toMatchSnapshot Co-authored-by: Bart Kalisz <bartlomiej.kalisz@gmail.com> --------- Co-authored-by: Glen Davies <glendaviesnz@users.noreply.github.com> Co-authored-by: Bart Kalisz <bartlomiej.kalisz@gmail.com>
What?
Upgrade Playwright to 1.30.0
Testing Instructions
CI should pass.