-
Notifications
You must be signed in to change notification settings - Fork 102
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
fix: migrate to EventEmitterAsyncResource from core #433
fix: migrate to EventEmitterAsyncResource from core #433
Conversation
17bc15d
to
c2fc550
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.
LGTM
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.
LGTM, but lint is failing; can you check?
I guess this loses the compatibility with node versions prior to inclusion of EventEmitterAsyncResource
That's fine, EventEmitterAsyncResource
was added on v16, which is the minimum version we support 👍
Thanks for the contribution! |
@metcoder95 may I ask you to create a new release for it? |
I'll reuse the current one 👍 |
Fixes: #426
EventEmitterAsyncResource
EventEmitterAsyncResource
Note: I guess this loses the compatibility with node versions prior to inclusion of
EventEmitterAsyncResource