Skip to content
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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add WHATWG Encoding API globals #833

Merged
merged 1 commit into from May 13, 2018

Conversation

@MarkTiedemann
Copy link
Contributor

MarkTiedemann commented May 13, 2018

TextEncoder and TextDecoder may become global variables in Node 11.

I don't expect this PR to be merged now, I'm just sending this as a tiny, early heads-up. 馃檪

Issue: nodejs/node#20365
PR: nodejs/node#20662

@MarkTiedemann MarkTiedemann mentioned this pull request May 13, 2018
0 of 4 tasks complete
Copy link
Contributor

cjihrig left a comment

LGTM, thanks!

@geek geek self-assigned this May 13, 2018
@geek geek added the bug label May 13, 2018
@geek geek added this to the 15.4.5 milestone May 13, 2018
@geek geek merged commit 3e4a37f into hapijs:master May 13, 2018
2 checks passed
2 checks passed
Node Security No known vulnerabilities found
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@geek

This comment has been minimized.

Copy link
Member

geek commented May 13, 2018

@MarkTiedemann and @cjihrig this is published as 15.4.5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can鈥檛 perform that action at this time.