Skip to content

Updating the "rush add " command with "--caret" flag in Contributing.md#4512

Merged
HarshaNalluru merged 1 commit intomasterfrom
HarshaNalluru-patch-4
Jul 30, 2019
Merged

Updating the "rush add " command with "--caret" flag in Contributing.md#4512
HarshaNalluru merged 1 commit intomasterfrom
HarshaNalluru-patch-4

Conversation

@HarshaNalluru
Copy link
Contributor

@HarshaNalluru HarshaNalluru commented Jul 29, 2019

Updating the "rush add " command with "--caret" flag.

Installing a new npm package -

  • "npm install" adds ^ by default in the package.json,
  • "rush add" adds ^ only when the --caret flag is specified[ in the package.json].

Example -

  • rush add -p hello --caret -> "hello": "^0.3.2"
  • rush add -p hello -> "hello": "~0.3.2"

@HarshaNalluru HarshaNalluru added the EngSys This issue is impacting the engineering system. label Jul 29, 2019
@HarshaNalluru HarshaNalluru self-assigned this Jul 29, 2019
@bsiegel
Copy link
Member

bsiegel commented Jul 30, 2019

Looks good to me. In the "Installing and managing dependencies" section we do document the recommendation to use the full syntax (rush add <packagename>@^<version>). This is a good addition to the table though. Thanks!

@HarshaNalluru
Copy link
Contributor Author

Yes, Brandon.
I've seen that after Mike pointed it out to me and we thought it would be better to have it in the table as well.

@HarshaNalluru HarshaNalluru merged commit fd1d601 into master Jul 30, 2019
@HarshaNalluru HarshaNalluru deleted the HarshaNalluru-patch-4 branch August 2, 2019 05:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

EngSys This issue is impacting the engineering system.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants