-
-
Notifications
You must be signed in to change notification settings - Fork 8k
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
Run correct npm completion script #427
Comments
You could create an alias, and put the path to the alias in your |
That's just another way of doing the same thing though, right? If I use a different version of node, I'm now using the wrong completion.sh. |
Ah, i see what you're saying. You could do |
Would adding a line to nvm use do the trick? Link for if I do it: https://github.com/creationix/nvm/blob/master/nvm.sh#L462 |
I don't think it's appropriate to handle shell completion inside |
Either through an option ( |
How big of an issue is this, since the npm completion script mostly just calls out to |
The idea, I think, is using the Shims don't actually end up improving anything, if Ruby's tools are any indication :-) |
This is really only an issue when switching between node versions that include wildly different npm versions. This can be solved by simply using the latest version of I personally just do the following:
If you've installed the latest npm but get an older version when you |
I don't have I highly discourage anyone from installing anything outside an |
|
Just like in #606, that's the sort of thing that |
Seems like a fairly rude thing to do 😒 - What other modules should I be afraid |
I haven't committed to doing that - I may instead just make it so |
Well, that would indeed be less terrible. |
On my Ubuntu 20.04 box with nvm 0.35.3 installed, there exists the environment variable NVM_BIN="/home/[username]/.nvm/versions/node/v12.16.3/bin" Therefore, a
does the trick version independent. |
I have the following line in my .bash_profile:
That seems kind of fragile. Is there a better way, or could a better way be written?
The text was updated successfully, but these errors were encountered: