-
Notifications
You must be signed in to change notification settings - Fork 1.2k
More dependency updates #10892
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
More dependency updates #10892
Conversation
63bcc2c to
8e38e16
Compare
8e38e16 to
012184c
Compare
a0e6d1a to
012184c
Compare
| const os = require('os'); | ||
|
|
||
| // For testing purposes: | ||
| const execa = { sync: execaSync }; |
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.
The corresponding tests work by overriding sync on an object.
Updating the tests would be more work I think.
| @@ -1,20 +0,0 @@ | |||
| 'use strict'; | |||
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.
This was not used consistently, and we can get the same or more info by setting NODE_DEBUG=execa when needed.
No description provided.