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

Replace pacote with npm-registry-fetch #1329

Merged
merged 20 commits into from
Mar 27, 2024
Merged

Replace pacote with npm-registry-fetch #1329

merged 20 commits into from
Mar 27, 2024

Conversation

raineorshine
Copy link
Owner

@raineorshine raineorshine commented Sep 13, 2023

Replace pacote with npm-registry-fetch to reduce the bundle size.

There is some risk that the request headers are not recreated accurately, so this should wait for a major version change to be safe.

@raineorshine raineorshine force-pushed the npm-registry-fetch branch 2 times, most recently from 0b5855d to 0113a20 Compare September 13, 2023 00:56
@raineorshine raineorshine force-pushed the main branch 4 times, most recently from 03526e5 to 66f2c08 Compare September 13, 2023 16:15
@raineorshine raineorshine added this to the v17 milestone Sep 16, 2023
@raineorshine raineorshine force-pushed the main branch 3 times, most recently from c05c239 to e416363 Compare October 4, 2023 21:50
@raineorshine raineorshine force-pushed the npm-registry-fetch branch 2 times, most recently from dd1eabf to 863c4c3 Compare November 3, 2023 15:55
@raineorshine raineorshine changed the base branch from main to v17-next November 3, 2023 15:55
@raineorshine

This comment was marked as outdated.

@raineorshine raineorshine changed the base branch from v17 to main March 12, 2024 20:13
@wmertens

This comment was marked as resolved.

@raineorshine raineorshine marked this pull request as ready for review March 27, 2024 21:52
@raineorshine raineorshine merged commit 8cf4dc9 into main Mar 27, 2024
8 checks passed
@raineorshine raineorshine deleted the npm-registry-fetch branch March 27, 2024 22:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants