-
Notifications
You must be signed in to change notification settings - Fork 1k
chore: roll to 1.53.1 #2892
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&rdqu 10000 o;, 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
chore: roll to 1.53.1 #2892
Conversation
playwright/_impl/_browser_type.py
Outdated
params["clientCertificates"] = await to_client_certificates_protocol( | ||
params["clientCertificates"] | ||
) | ||
params["selectorEngines"] = self._playwright.selectors._selectorEngines |
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.
This is copied from _browser.py
, with the changes being this line and below.
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.
I'd like a follow up with an explicit timeout in each method, instead of a global "timeout calculator". It feels like that would be less magic and not so bad in terms of code.
Originally mirrored microsoft/playwright-dotnet#3184, but due to the selector bug (microsoft/playwright#36227) causing half the Python tests to fail, the roll had to be made to include this change. Thus the roll is the entire roll to
1.53.1
.Implements changes from microsoft/playwright#35888, microsoft/playwright#35969, and microsoft/playwright#35988, among others.
References #2818