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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(v8.x backport) test: use ES6 classes instead of util.inherits #16946

Closed

Conversation

tniessen
Copy link
Member

This is an early backport of #16938 for node 8.x.

Affected core subsystem(s)

test

@tniessen tniessen added refactor to ES6+ test Issues and PRs related to the tests. labels Nov 11, 2017
@nodejs-github-bot nodejs-github-bot added test Issues and PRs related to the tests. v8.x labels Nov 11, 2017
PR-URL: nodejs#16938
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
@tniessen tniessen force-pushed the backport-16938-to-v8.x branch from 9a02f6a to 2291b3e Compare November 14, 2017 15:52
gibfahn pushed a commit that referenced this pull request Nov 14, 2017
PR-URL: #16938
Backport-PR-URL: #16946
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
@gibfahn
Copy link
Member

gibfahn commented Nov 14, 2017

Landed in 981a1ef

@gibfahn gibfahn closed this Nov 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test Issues and PRs related to the tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants