-
Notifications
You must be signed in to change notification settings - Fork 47
Instruction for installation Node on Mac #85
Conversation
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.
Thanks for opening the PR! 🎉
As this is one of several ways to install Node.js, the others being using the binary or nvm (Node Version Manager), I'd recommend specifying that this method uses Brew in the title of the PR.
For example:
Instruction for installation Node on Mac using Brew
- **PR:** | ||
- **Issue:** [#60](https://github.com/nodejs/website-redesign/issues/60) | ||
- **Keywords:** install node mac | ||
- **Summary:** Instruction how to install node.js on MacOS using Homebrew |
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 would highly recommend not doing this. The major problem with this type of installation is that it would require sudo
to install packages. Let's recommend nvm or nvs.
For reference, here's another article that approaches similar content – there's a lot of ideas that can be recycled 😄 https://nodesource.com/blog/installing-nodejs-tutorial-mac-os-x/ |
@stefanovich-yulia I'm happy to help guide you through completing this PR. I'd recommend reading Matteo's review above and reviewing Tierney's suggested article. Ping me by @ mentioning me in a comment and I'll help walk you through revising the PR so we can merge it. Or swing by the next Nodeschool meetup. |
This has been stalled... can I go on with using @bnb approaches to get it merged? |
Hi,
Sorry I missed everything somehow.
Do you need any actions from me now?
…On Thu, Jan 24, 2019 at 7:31 AM Agiri Abraham Jr ***@***.***> wrote:
This has been stalled... can I go on with using @bnb
<https://github.com/bnb> approaches to get it merged?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#85 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AKa8ZAqHMEot6H6QuCXwSfrKGGd8smxwks5vGdGygaJpZM4WCvms>
.
|
@stefanovich-yulia a previous comment need to be resolved before this can be merged. This comment suggests how you can resolve that. Mentors at NodeSchoolYVR can help you resolve the comments at the next session too 😄 |
I've unarchived this repo so I can close all PRs and issues before re-archiving. |
No description provided.