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

Update install.md #4611

Merged
merged 2 commits into from May 2, 2024
Merged

Update install.md #4611

merged 2 commits into from May 2, 2024

Conversation

jlguenego
Copy link
Contributor

replaced --location=global with -g

replaced --location=global with -g
@@ -32,7 +32,7 @@ Learn the basics before getting started, how to install, where is the location o
Using `npm`

```bash
npm install --location=global verdaccio@next-7
npm install -g verdaccio@next-7
Copy link
Member

@juanpicado juanpicado May 1, 2024

Choose a reason for hiding this comment

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

🤔 I'm pretty sure the --location was quite new but I don't find it anymore at the docs. You might want to also update other places like the README which also we have that location flag.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

--location is for npm config set (or npm set).
Not for install 😊

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I did the modification for the other place. There is another commit in this pull request.

@juanpicado juanpicado added the docs label May 1, 2024
Copy link
Member

@juanpicado juanpicado left a comment

Choose a reason for hiding this comment

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

Thanks @jlguenego

@jlguenego
Copy link
Contributor Author

jlguenego commented May 2, 2024 via email

@juanpicado juanpicado merged commit 3293c9a into verdaccio:master May 2, 2024
26 checks passed
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.

None yet

2 participants