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

docs(scope): update example command to work in windows #7247

Merged
merged 1 commit into from
Feb 26, 2024

Conversation

robertobasile84
Copy link
Contributor

Missing = was causing error

References

Missing = was causing error
@robertobasile84 robertobasile84 requested a review from a team as a code owner February 25, 2024 20:45
@wraithgar
Copy link
Member

This should work fine

~ $ npm config set @foo:registry https://reg.example.com
~ $ grep foo ~/.npmrc
@foo:registry=https://reg.example.com

@wraithgar wraithgar closed this Feb 26, 2024
@robertobasile84
Copy link
Contributor Author

@wraithgar did you try this on windows?

image

@wraithgar
Copy link
Member

Try putting the url in quotes

@robertobasile84
Copy link
Contributor Author

robertobasile84 commented Feb 26, 2024

image

Same with " instead of '

If my suggestion works also in Linux system, I think it should be suggested singe it works everywhere

@wraithgar wraithgar changed the title Update scope.md docs(scope): update example command to work in windows Feb 26, 2024
@wraithgar wraithgar reopened this Feb 26, 2024
@wraithgar wraithgar merged commit 686a622 into npm:latest Feb 26, 2024
9 checks passed
@github-actions github-actions bot mentioned this pull request Feb 26, 2024
@robertobasile84 robertobasile84 deleted the patch-2 branch February 26, 2024 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants