-
-
Notifications
You must be signed in to change notification settings - Fork 8.6k
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
uppercase in .nvmrc are handled differently for use
and install
commands
#3417
Comments
It indeed should only be lowercase. how did the uppercase version end up in .nvmrc? |
I guess nobody cared and eventually it got merged like this 😄 |
so, i'm not sure what nvm should be doing here - |
I see two possibilities:
|
The first bullet point is the case, but that does also mean nvm validates it ( Certainly nvm could be case-insensitive with its LTS aliases, but that’s a lot of complexity and implicitness for an exceedingly uncommon case (and now nvm has --save, so hand-written nvmrc files aren’t needed as much) |
rereading the OP, the bug seems to be that |
New Stuff - `install.sh`: add `$ZDOTDIR` to zsh search (#3458) Fixes - `reinstall-packages`: do not reinstall corepack (#3544) - avoid bash-specific syntax (#3499) - `install-latest-npm`: npm v11 is out - `nvm_install_latest_npm`: avoid unbound variable (#3447) - give a more helpful message when `lts` alias is mistakenly used (#3441) - `nvm ls`, `nvm alias`, `nvm install`: error when an LTS name is invalid - `nvm_normalize_lts`: error when an LTS name is not lowercase (#3417) Documentation - [readme] update link - [readme] fix `--no-use` example (#3479) - [readme] update copyright notice (#3507) - [readme] note zsh-nvm's AUTO_USE option (#2670) - [readme] add note about reloading zshrc after editing (#3052) - [readme] Update shell profile file install notes (#2241) - [readme] add docker tips (#2318) - [readme] remove `avn` from readme (#3469) - [readme] fnm -> nvm.fish (#2541) Refactors - prefer `case` over if/else chains - combine `sed -e` invocations/arguments Tests - `nvm exec`/`nvm run`: add `--silent` tests (#1259) - [actions] release test needs git tags - migrate `installation_iojs` test suite to GitHub Actions (#3476) - Migrate slow test suite from Travis CI (#3470) - temporarily skip this failing travis test to unblock progress - [actions] TOC: use latest LTS node - `install.sh`: clean up `nvm_detect_profile` tests - `nvm_detect_profile`: refactor (#3467) - run urchin tests on pull requests (#3466) - update mocks - ensure that unit tests use only mocked LTS names - [actions] use `node/install` instead of `node/run` Meta - disable blank issues - update issue template - add DCO (#3456) - Rename .github/ISSUE_TEMPLATE.md to .github/ISSUE_TEMPLATE/ISSUE_TEMPLATE.md (#3454)
Operating system and version:
Linux (archlinux)
nvm debug
output:nvm ls
output:How did you install
nvm
?Archlinux AUR (community package registry) using an AUR helper (
yay
in my case`What steps did you perform?
.nvmrc
file which contains the followinglts/Iron
nvm use
What happened?
nvm use
commands output an error telling "lts/Iron" is not yet installedbut a corresponding v20 node is already installed, but still running
nvm install
to be surenvm install
finds the corresponding installed version, and use itWhat did you expect to happen?
Expected the
nvm use
to find the already installed version, likenvm install
did, and use it.To be fair, the workaround is simply to fix the uppercase in the .nvmrc file (from
lts/Iron
tolts/iron
), I just wanted to create the issue if other stumble accross the same behavior.The text was updated successfully, but these errors were encountered: