-
Notifications
You must be signed in to change notification settings - Fork 6.1k
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
fix: change NVM to nvm #6722
fix: change NVM to nvm #6722
Conversation
I believe `nvm` is always lowercase as seen in their readme https://github.com/nvm-sh/nvm/blob/master/README.md Signed-off-by: Steven <steven@ceriously.com>
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Lighthouse Results
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM ! Just, is there any other mention of nvm that might be in the wrong case?
Good point, I just pushed another commit (f415c4d) where the code comment was using the wrong case as well. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fast-tracking this small change
Description
I believe
nvm
is always lowercase as seen in their readmehttps://github.com/nvm-sh/nvm/blob/master/README.md
Validation
Visit https://nodejs-org-git-fork-styfle-patch-1-openjs.vercel.app/en/download/package-manager and note the dropdown.
Related Issues
None
Check List
npm run format
to ensure the code follows the style guide.npm run test
to check if all tests are passing.npx turbo build
to check if the website builds without errors.