Skip to content

Update Dependencies - #6133

Merged
lukekarrys merged 21 commits into
latestfrom
lk/deps
Feb 7, 2023
Merged

Update Dependencies#6133
lukekarrys merged 21 commits into
latestfrom
lk/deps

Conversation

@lukekarrys

Copy link
Copy Markdown
Contributor

No description provided.

@npm-cli-bot

npm-cli-bot commented Feb 6, 2023

Copy link
Copy Markdown
Collaborator

no statistically significant performance changes detected

timing results
app-largecleanlock-onlycache-onlycache-only
peer-deps
modules-onlyno-lockno-cacheno-modulesno-cleanno-clean
audit
npm@846.849 ±6.6221.484 ±0.0719.754 ±0.2923.197 ±0.283.732 ±0.013.821 ±0.013.056 ±0.0414.644 ±0.023.007 ±0.024.278 ±0.02
#613340.346 ±5.1620.920 ±0.2019.337 ±0.5222.514 ±0.773.571 ±0.063.573 ±0.042.870 ±0.0614.109 ±0.152.743 ±0.034.386 ±0.00
app-mediumcleanlock-onlycache-onlycache-only
peer-deps
modules-onlyno-lockno-cacheno-modulesno-cleanno-clean
audit
npm@831.585 ±1.8817.018 ±0.1715.287 ±0.0716.368 ±0.663.308 ±0.013.363 ±0.063.000 ±0.0410.493 ±0.162.739 ±0.033.862 ±0.03
#613328.884 ±0.5816.615 ±0.1315.633 ±0.1116.861 ±0.483.329 ±0.043.342 ±0.042.784 ±0.0210.738 ±0.052.644 ±0.113.701 ±0.00

Comment threadworkspaces/arborist/test/fixtures/server.js Fixed
Comment threadworkspaces/arborist/test/fixtures/server.js Fixed
Comment threadworkspaces/arborist/test/fixtures/server.js Fixed
@lukekarrys
lukekarrys marked this pull request as ready for review February 7, 2023 00:24
@lukekarrys
lukekarrys requested a review from a team as a code ownerFebruary 7, 2023 00:24
@lukekarrys
lukekarrys requested review from nlf and removed request for a teamFebruary 7, 2023 00:24

@wraithgarwraithgar left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only two deps are hoisted differently than before, and the hoisting is the correct choice in those cases: minimatch and fs-minipass.

In both cases the newest version is hoisted, and the version that is hoisted is used as a production dep.

All other changes are simply version updates.

@lukekarrys
lukekarrys merged commit 8200f4f into latestFeb 7, 2023
@lukekarrys
lukekarrys deleted the lk/deps branch February 7, 2023 16:31
@github-actionsgithub-actionsBot mentioned this pull request Feb 7, 2023
@github-actionsgithub-actionsBot mentioned this pull request Oct 6, 2023
@github-actionsgithub-actionsBot mentioned this pull request Feb 26, 2024
Sign up for freeto 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

@lukekarrys@npm-cli-bot@wraithgar@github-advanced-security