-
Notifications
You must be signed in to change notification settings - Fork 331
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
Add Browser Rendering JS wrapped binding #3334
Conversation
907c834
to
8e093a9
Compare
8e093a9
to
b2ad485
Compare
b2ad485
to
1737772
Compare
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.
Code changes LGTM
@G4brym Would you mind updating the pull-request description? |
Just updated the title and description, let me know if there's anything else for me to do 👍 |
Head branch was pushed to by a user without write access
1737772
to
07d73be
Compare
Sorry the builds were failing with the previous compability date, pushing a fix disabled auto merge |
Browser rendering team is migrating from a simple Fetcher binding into a JS wrapped binding.
This pull requests adds a very simple wrapped binding that only exposes the already existing
.fetch()
method for existing worker.Follow up pr's will include adding custom methods to this new binding once it starts being used.