-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
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
Installation instructions are unclear for most people (Discussion) #748
Comments
You don't, typically. It depends on the specific array of profile files that each user has on their system. I'd love to fix it, but non-interactive profile detection is insanely complex. I'll certainly be happy to accept README PRs like #704, but I'd rather figure out a better way to properly do profile file detection. Another solution is to modify all profile files, but that seems dangerous because, for example, I have my |
I see. I was looking for the issue #704, but couldn't find it via search, because the wording was different. I'll continue the discussion there! |
reflect the need to optionally source nvm after curl based installation. nvm-sh#704 nvm-sh#748 and http://stackoverflow.com/a/17707224/690866
This is in response to a popular stack overflow answer that has helped people resolve the "command not found" hiccup: https://stackoverflow.com/a/17707224/2031033 Also mentioned here: nvm-sh#748
This is in response to a popular stack overflow answer that has helped people resolve the "command not found" hiccup: https://stackoverflow.com/a/17707224/2031033 Also mentioned here: nvm-sh#748
Hello everyone!
I feel like the documentation should make it clearer that you need to run
source ~/.nvm/nvm.sh
after installing if you use zsh.Many people are running the install script and are then wondering why they get a
zsh: command not found: nvm
error. In fact, over 10000 people had to consult Stackoverflow to get nvm running including myself, I'd imagine that there are even more people that are not using nvm at all because of this.Do you agree that we should make it clearer?
If so, what changes should we make specifically? Let's discuss.
Thanks for the really awesome software.
Best,
Jonny
The text was updated successfully, but these errors were encountered: