-
-
Notifications
You must be signed in to change notification settings - Fork 6
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
Use os.userInfo().shell
#3
Labels
Comments
From the docs, |
It will only be used on non-Windows systems. |
Awesome! I can make the PR using the latest API. Does this mean you're
dropping support for Node `v4`?
--
Kunall Banerjee
|
No, not yet. This is just a reminder for when I do. Node.js 4 is still LTS. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Sometime in the future when we can target Node.js 6.
nodejs/node#6104
I have a polyfill, but not worth the overhead.
The text was updated successfully, but these errors were encountered: