Skip to content

docs: recommend npm ci#371

Closed
privatenumber wants to merge 1 commit into
actions:mainfrom
privatenumber:patch-1
Closed

docs: recommend npm ci#371
privatenumber wants to merge 1 commit into
actions:mainfrom
privatenumber:patch-1

Conversation

@privatenumber

Copy link
Copy Markdown

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
Copy Markdown
Contributor

This is already addressed in this PR as well #326

@dmitry-shibanov

Copy link
Copy Markdown
Contributor

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

@BigsteppA1

Copy link
Copy Markdown

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.

4 participants