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

Fix npm install "--saFe" option typo in README.md #48

Merged
merged 1 commit into from Sep 6, 2017

Conversation

Projects
None yet
2 participants
@lostintime
Contributor

lostintime commented Sep 6, 2017

--safe option looks like a typo, as far as I know.

$ npm --version
3.10.10

$ npm install --help

npm install (with no args, in package dir)
npm install [<@scope>/]<pkg>
npm install [<@scope>/]<pkg>@<tag>
npm install [<@scope>/]<pkg>@<version>
npm install [<@scope>/]<pkg>@<version range>
npm install <folder>
npm install <tarball file>
npm install <tarball url>
npm install <git:// url>
npm install <github username>/<github project>

aliases: i, isntall
common options: [--save|--save-dev|--save-optional] [--save-exact]

Also --save option is replaced by --save-prod in npm version 5 (maybe earlier) which is the default.

$ npm --version
5.4.0

$ npm install --help

npm install (with no args, in package dir)
npm install [<@scope>/]<pkg>
npm install [<@scope>/]<pkg>@<tag>
npm install [<@scope>/]<pkg>@<version>
npm install [<@scope>/]<pkg>@<version range>
npm install <folder>
npm install <tarball file>
npm install <tarball url>
npm install <git:// url>
npm install <github username>/<github project>

aliases: i, isntall, add
common options: [--save-prod|--save-dev|--save-optional] [--save-exact] [--no-save]
@codecov

This comment has been minimized.

Show comment
Hide comment
@codecov

codecov bot Sep 6, 2017

Codecov Report

Merging #48 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #48   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          24     24           
  Lines        2470   2470           
  Branches      393    393           
=====================================
  Hits         2470   2470

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 31794a4...868ce48. Read the comment docs.

codecov bot commented Sep 6, 2017

Codecov Report

Merging #48 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #48   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          24     24           
  Lines        2470   2470           
  Branches      393    393           
=====================================
  Hits         2470   2470

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 31794a4...868ce48. Read the comment docs.

@alexandru

This comment has been minimized.

Show comment
Hide comment
@alexandru

alexandru Sep 6, 2017

Member

Thanks @lostintime, indeed it's a typo, too much copy/pasting 😊

Member

alexandru commented Sep 6, 2017

Thanks @lostintime, indeed it's a typo, too much copy/pasting 😊

@alexandru alexandru merged commit 3e75157 into funfix:master Sep 6, 2017

3 checks passed

codecov/patch Coverage not affected when comparing 31794a4...868ce48
Details
codecov/project 100% remains the same compared to 31794a4
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

alexandru added a commit that referenced this pull request Sep 6, 2017

alexandru added a commit that referenced this pull request Sep 6, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment