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

pacote@10.1.1 #280

Closed
wants to merge 1 commit into from
Closed

pacote@10.1.1 #280

wants to merge 1 commit into from

Conversation

kibertoad
Copy link

What / Why

Update pacote dependency to fix peerDependenciesMeta support.

References

@kibertoad kibertoad requested a review from a team as a code owner November 3, 2019 20:20
@kibertoad
Copy link
Author

@isaacs Build errors out after update with

npm ERR! code MODULE_NOT_FOUND
npm ERR! Cannot find module 'pacote/extract'

Is there a new external dependency on the project? Any suggestions on how to fix?

@kibertoad
Copy link
Author

@mikemimik Looks like I'm supposed to commit in updated dependencies in node_modules. Could you advise whether I should just commit everything that changed, or there are more specific rules for that?

@isaacs
Copy link
Contributor

isaacs commented Nov 4, 2019

Pacote 10 is a breaking change. (Several, in fact.

This is fixed in 9.5.9, already set to release in npm 6.13.0, in two days.

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.

2 participants