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.

Sign up for GitHub

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

Jump to bottom

docs(npm): update npm docs #2498

Merged
merged 1 commit into from Jan 21, 2021
Merged

docs(npm): update npm docs #2498

merged 1 commit into from Jan 21, 2021

Conversation

darcyclarke
Copy link
Contributor

@darcyclarke darcyclarke added Release 7.x work is associated with a specific npm 7 release release: next These items should be addressed in the next release labels Jan 15, 2021
@darcyclarke darcyclarke requested a review from a team as a code owner January 15, 2021 21:26
@npm-deploy-user
Copy link

npm-deploy-user commented Jan 15, 2021 •
edited

found 1 benchmarks with statistically significant performance improvements

  • app-large: cache-only:legacy-peer-deps
timing results
app-large clean lock-only cache-only cache-only
legacy-peer-deps		 modules-only no-lock no-cache no-modules no-clean no-clean
audit
npm@7 47.909 ±0.20 30.632 ±0.17 31.313 ±1.12 29.879 ±0.26 2.472 ±0.04 2.492 ±0.01 1.738 ±0.03 13.127 ±0.09 1.727 ±0.05 3.722 ±0.07
#2498 46.035 ±0.10 31.015 ±0.03 29.739 ±1.21 26.873 ±0.55 2.491 ±0.02 2.512 ±0.04 1.771 ±0.06 13.121 ±0.13 1.786 ±0.03 3.698 ±0.01
app-medium clean lock-only cache-only cache-only
legacy-peer-deps		 modules-only no-lock no-cache no-modules no-clean no-clean
audit
npm@7 34.159 ±0.94 21.412 ±0.08 21.359 ±0.80 20.720 ±0.13 2.356 ±0.05 2.428 ±0.06 1.633 ±0.03 8.228 ±0.02 1.669 ±0.06 3.004 ±0.06
#2498 32.537 ±0.74 22.006 ±0.10 19.251 ±0.80 19.588 ±0.55 2.426 ±0.03 2.383 ±0.00 1.643 ±0.03 8.344 ±0.12 1.688 ±0.01 3.153 ±0.23

@darcyclarke darcyclarke added the semver:patch semver patch level for changes label Jan 21, 2021
@darcyclarke @ruyadorno
docs(npm): update npm docs
ec1f06d 
- fix reference to issue tracker (ie. fixes #1507)
- collapse contributing guideline ref
- remove author-specific information (still exists in `package.json`)

PR-URL: #2498
Credit: @darcyclarke
Close: #2498
Reviewed-by: @ruyadorno
@ruyadorno ruyadorno changed the base branch from latest to release/v7.4.3 January 21, 2021 16:17
@ruyadorno ruyadorno merged commit ec1f06d into release/v7.4.3 Jan 21, 2021
This was referenced Jan 21, 2021
Merged
Closed
@nlf nlf deleted the darcyclarke-patch-npm-docs branch March 28, 2022 16:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wraithgar wraithgar wraithgar approved these changes

Assignees
No one assigned
Labels
release: next These items should be addressed in the next release Release 7.x work is associated with a specific npm 7 release semver:patch semver patch level for changes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

fix community/bug reporting guidance in docs/content/cli-commands/npm.md
4 participants
@darcyclarke @npm-deploy-user @wraithgar @ruyadorno