-
Notifications
You must be signed in to change notification settings - Fork 29.7k
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
doc: indicate the format of process.version #34872
Conversation
I'd need to check, but I think that was intentional. |
If my recollection is correct, it may be worthwhile to have a discussion about documentation-deprecating |
Or that conversation might be made unnecessary by the change here. Maybe it's enough to direct people to the right thing. |
Landed in 0dae5d9 |
PR-URL: #34872 Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Rich Trott <rtrott@gmail.com>
PR-URL: #34872 Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Rich Trott <rtrott@gmail.com>
PR-URL: #34872 Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Rich Trott <rtrott@gmail.com>
PR-URL: #34872 Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Rich Trott <rtrott@gmail.com>
I needed to parse
process.version
and didn't know that it is prefixed with av
until I inspected the value.As an aside, it seems inconsistent to me that
process.version
is prefixed with av
, butprocess.versions.node
isn't. Would it be worth opening an issue to discuss that?Checklist
make -j4 test
(UNIX), orvcbuild test
(Windows) passes