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: recommend npm ci #371

Closed
wants to merge 1 commit into from
Closed

docs: recommend npm ci #371

wants to merge 1 commit into from

Conversation

privatenumber
Copy link

Description:
Docs to use npm ci instead of npm install.

According to npm docs:

npm ci
This command is similar to npm install, except it's meant to be used in automated environments such as test platforms, continuous integration, and deployment -- or any situation where you want to make sure you're doing a clean install of your dependencies.

npm ci will be significantly faster when:

  • There is a package-lock.json or npm-shrinkwrap.json file.
  • The node_modules folder is missing or empty.

In particular, the benefit of using npm ci over npm install is that there will not by any dependency discrepancy because it installs the exact dependency versions from lock. In contrast, npm install will install any new version that fits into the semver range defined in package.json, which can yield to unexpected behavior or bugs.

Related issue:
Add link to the related issue.

Check list:

  • Mark if documentation changes are required.
  • Mark if tests were added or updated to cover the changes.

@jonkoops
Copy link
Contributor

This is already addressed in this PR as well #326

@dmitry-shibanov
Copy link
Contributor

Hello @privatenumber. I'm going to close the pull request as duplicate of #326.

@BigsteppA1
Copy link

logo

deining pushed a commit to deining/setup-node that referenced this pull request Nov 9, 2023
…ions#371)

Bumps [eslint-config-prettier](https://github.com/prettier/eslint-config-prettier) from 8.4.0 to 8.5.0.
- [Release notes](https://github.com/prettier/eslint-config-prettier/releases)
- [Changelog](https://github.com/prettier/eslint-config-prettier/blob/main/CHANGELOG.md)
- [Commits](prettier/eslint-config-prettier@v8.4.0...v8.5.0)

---
updated-dependencies:
- dependency-name: eslint-config-prettier
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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.

None yet

4 participants