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
chore: use node_bindings loop for clarity #24391
Conversation
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.
is the backport necessary ?
@deepak1556 backporting change like this reduces code drift and makes it easier for trop to succeed, so i normally try where possible :) |
No Release Notes |
I have automatically backported this PR to "10-x-y", please check out #24416 |
I have automatically backported this PR to "8-x-y", please check out #24417 |
I have automatically backported this PR to "9-x-y", please check out #24418 |
Tôi không nhận được được thông báo mật khẩu và số sim của tôi cũng bị khóa |
Description of Change
Clarify loop being used by always passing in
node_bindings_->uv_loop()
, as it will beuv_default_loop
unless we're in aWebWorkerObserver
. This change is for consistency and should affect no behavior.cc @ckerr @zcbenz
Checklist
npm test
passesRelease Notes
Notes: none