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
Revert "Fix definition of URL constructor (#5521)" #5564
Conversation
Ok, I understood. I hope you remember to include this in v1.1.0. |
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.
Ok, but it seems there's intermittent CI failure
This reverts commit 63bc468.
It is actually a breaking change with |
Attempt 2. See denoland#5521 and denoland#5564.
…land/deno#5564) This reverts commit 63bc468.
…land/deno#5564) This reverts commit 63bc468.
…land/deno#5564) This reverts commit 63bc468.
…land/deno#5564) This reverts commit 63bc468.
…land/deno#5564) This reverts commit 63bc468.
…land/deno#5564) This reverts commit 63bc468.
…land/deno#5564) This reverts commit 63bc468.
…land/deno#5564) This reverts commit 63bc468.
…land/deno#5564) This reverts commit 63bc468.
I landed this change without thinking about it too much. This is an interface change - it actually breaks Oak. We should not land it in the next patch release (1.0.1). I think we should land it in 1.1.0.
This reverts commit 63bc468.
cc @3846masa