-
Notifications
You must be signed in to change notification settings - Fork 29.6k
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
Update the Readme #2361
Comments
@jasnell I'm working on his. |
Question - do we want to update the references to io.js within code? Seems that it would be more correct, but I know extra noise in code is frowned upon. |
In general, it would likely be a good idea. Rip the bandaid off all at one so to speak. |
👍 On |
cjihrig
added a commit
to cjihrig/node
that referenced
this issue
Aug 23, 2015
This commit replaces instances of io.js with Node.js, based on the recent convergence. There are some remaining instances of io.js, related to build and the installer. Fixes: nodejs#2361 PR-URL: nodejs#2367 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: João Reis <reis@janeasystems.com>
cjihrig
added a commit
to cjihrig/node
that referenced
this issue
Aug 23, 2015
This commit sets the value of process.release.name to "node". Fixes: nodejs#2361 PR-URL: nodejs#2367 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Rod Vagg <rod@vagg.org>
cjihrig
added a commit
to cjihrig/node
that referenced
this issue
Aug 23, 2015
Fixes: nodejs#2361 PR-URL: nodejs#2367 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Rod Vagg <rod@vagg.org>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Obviously, the project README.md needs to be updated now 👍 Can we go ahead and get things renamed?
The text was updated successfully, but these errors were encountered: