-
-
Notifications
You must be signed in to change notification settings - Fork 143
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
Roll to Playwright v1.36.1 #358
Conversation
CONTRIBUTING.md
Outdated
1. Fix merge conflicts if any, otherwise ignore this step. Once you are happy you can commit the changes `cd playwright; git commit -am "apply patch" && cd ..` | ||
1. Regenerate a new patch `bash scripts/update-patch.sh` | ||
1. Generate go code `go generate ./...` | ||
2. Download current version of Playwright driver `go run scripts/install-browsers/main.go` |
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.
nit: its kinda nice to keep 1. everywhere, since then the numbers get auto-generated and its a smaller diff when steps are added or removed.
tests/page_test.go
Outdated
require.Equal(t, val, 42) | ||
val, err = page.Evaluate(`a => a`, 17) | ||
require.NoError(t, err) | ||
require.Equal(t, val, 17) |
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.
sounds like there is this? https://pkg.go.dev/math/big
Hi @mxschmitt , this pr rolling to v1.36.1.
Currently there is no automatic generation of
scripts/data/interfaces.json
, butvalidate-interfaces.js
has been added togenerate-api.sh
. This ensures that unimplemented interfaces are all explicitly ignored invalidate-interfaces.js
.