-
Notifications
You must be signed in to change notification settings - Fork 12.5k
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
Add extensions to bin
scripts.
#39380
Conversation
Adption to changes in Node. See nodejs/node#31415
I'm a bit wary that the |
It's a good point. Can i change PR and add "twins"? |
Appears to be a fix for #39381 |
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.
I can't think of how this could break something, so if another TS member feels good with it then we should be good to merge
We... don't set |
@weswigham effectively OP is experiencing this issue? nodejs/node#33226 I don't think we should merge anything with nonzero risk in order to enable experimental node stuff |
Yep, that looks to be a report of the same thing. |
This makes sense, we should probably close this. |
Adption to changes in Node.
See nodejs/node#31415
Fixes #