Skip to content
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

Add homebrew install instructions for nvm compat #516

Merged
merged 3 commits into from
Jun 5, 2017
Merged

Add homebrew install instructions for nvm compat #516

merged 3 commits into from
Jun 5, 2017

Conversation

ntwb
Copy link
Contributor

@ntwb ntwb commented May 29, 2017

@@ -9,6 +9,12 @@ This will also install Node.js if it is not already installed.
brew install yarn
```

If you use [nvm](https://github.com/creationix/nvm) you should exclude installing NodeJS so that the nvm binaries are used.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe change "nvm binaries" to "nvm's version of Node.js"?

Also, small nit: Use "Node.js", not "NodeJS".

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ha, and done, I've been writing it that way for years, my bad, sorry Node.js

@Daniel15
Copy link
Member

Daniel15 commented Jun 5, 2017

Thanks!

@Daniel15 Daniel15 merged commit 66c029d into yarnpkg:master Jun 5, 2017
@ntwb ntwb deleted the patch-1 branch June 5, 2017 07:27
@nandorojo
Copy link

Is this no longer published on the docs? Looks like the current installation explanation is a little less clear https://classic.yarnpkg.com/en/docs/install/#mac-stable

Thanks!

@Daniel15
Copy link
Member

@nandorojo Homebrew removed the ability to install Yarn without installing Node.js a while back. In fact they removed all customization options for formulae.

@nandorojo
Copy link

You’re right. I went with the CURL install instead — this may be the best option for nvm users to avoid node installs. Maybe I could add a PR to suggest that?

@Daniel15
Copy link
Member

Sure, feel free to submit a PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Document how to install yarn when you use nvm
3 participants