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

[readme] rearrange install args #3222

Merged
merged 1 commit into from
Dec 5, 2023

Conversation

chrismeyers
Copy link
Contributor

I was having trouble installing a new version of Node using the --latest-npm flag. After reading the help message again, I noticed the following:

The following optional arguments, if provided, must appear directly after `nvm install`

All of the nvm install examples in the README currently put the arguments after the version. This change should reduce confusion and better align with the intended usage.

Note: The --reinstall-packages-from flag works just fine when put after the version, but --latest-npm (and possibly others), need to come directly after nvm install.

Copy link
Member

@ljharb ljharb left a comment

Choose a reason for hiding this comment

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

Sure, thanks

@ljharb ljharb force-pushed the chore/rearrange-install-args branch from c7936cf to 5210304 Compare November 1, 2023 22:04
@ljharb ljharb changed the title [Docs]: Rearrange install args [readme] rearrange install args Nov 1, 2023
@ljharb ljharb force-pushed the chore/rearrange-install-args branch from 5210304 to 9c7bf2c Compare November 1, 2023 22:12
@ljharb ljharb force-pushed the chore/rearrange-install-args branch from 9c7bf2c to 6262b5a Compare December 5, 2023 04:46
@ljharb ljharb merged commit 6262b5a into nvm-sh:master Dec 5, 2023
92 of 109 checks passed
@chrismeyers chrismeyers deleted the chore/rearrange-install-args branch December 5, 2023 23:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants