Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

Instruction for installation Node on Mac #85

Closed
wants to merge 2 commits into from

Conversation

stefanovich-yulia
Copy link

No description provided.

@keywordnew keywordnew added the getting-started-docs Getting started documentation page label Aug 18, 2018
Copy link
Contributor

@keywordnew keywordnew left a 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
Copy link
Member

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.

@bnb
Copy link
Contributor

bnb commented Sep 13, 2018

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/

@amiller-gh amiller-gh mentioned this pull request Sep 13, 2018
7 tasks
@keywordnew
Copy link
Contributor

@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.

@codeekage
Copy link
Contributor

This has been stalled... can I go on with using @bnb approaches to get it merged?

@stefanovich-yulia
Copy link
Author

stefanovich-yulia commented Jan 24, 2019 via email

@keywordnew
Copy link
Contributor

@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 😄

@Trott
Copy link
Member

Trott commented Apr 22, 2023

I've unarchived this repo so I can close all PRs and issues before re-archiving.

@Trott Trott closed this Apr 22, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
getting-started-docs Getting started documentation page
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants