Skip to content
This repository was archived by the owner on Aug 11, 2022. It is now read-only.

Add a note to install --production. #19950

Merged
merged 1 commit into from
Mar 8, 2018
Merged

Add a note to install --production. #19950

merged 1 commit into from
Mar 8, 2018

Conversation

kyranet
Copy link
Contributor

@kyranet kyranet commented Mar 1, 2018

The npm install --production CLI command was quite confusing when it comes to the difference between installing the dependencies for a project and for adding a dependency to a project. This PR adds a note about it specifying the flag is only meaningful in the former case.

@kyranet kyranet requested a review from a team as a code owner March 1, 2018 11:24
Copy link
Contributor

@zkat zkat left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for this clarification! I'm sure it'll help clarify things for folks trying to make sense of all our flags :)

@zkat zkat changed the base branch from latest to release-next March 6, 2018 01:50
@zkat zkat merged commit 5876f3f into npm:release-next Mar 8, 2018
@kyranet kyranet deleted the patch-1 branch March 8, 2018 13:51
zkat pushed a commit that referenced this pull request Mar 8, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants