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

"npm -i zundo" fails #50

Closed
ohthepain opened this issue Sep 23, 2022 · 3 comments · Fixed by #51
Closed

"npm -i zundo" fails #50

ohthepain opened this issue Sep 23, 2022 · 3 comments · Fixed by #51

Comments

@ohthepain
Copy link

ohthepain commented Sep 23, 2022

Same results with "npm -i zundo@beta"

It seems to be requiring zustand 3.0.0. I am on zustand 4.1.1.

zundo depends on zustand 4.1.1 here: https://codesandbox.io/s/zundo-2dom9?file=/src/App.tsx:27-31

master* $ npm i zundo [9:46:00]
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving: powermad@0.1.0
npm ERR! Found: zustand@4.1.1
npm ERR! node_modules/zustand
npm ERR! zustand@"^4.1.1" from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer zustand@"^3.0.0" from zundo@2.0.0-beta.3
npm ERR! node_modules/zundo
npm ERR! zundo@"*" from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.

@ohthepain ohthepain changed the title "npm -I zundo" fails "npm -i zundo" fails Sep 23, 2022
@ohthepain
Copy link
Author

I confirmed in a new project. npm I undo works as long as I have not installed zustand

@charkour
Copy link
Owner

Oops, thanks an issue! I should be able to get that fixed soon.

Thanks for reporting!

@charkour charkour mentioned this issue Sep 23, 2022
@charkour
Copy link
Owner

This should be fixed.

npm i zundo@beta should update you to beta.4

Please let me know if you have any issues.

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 a pull request may close this issue.

2 participants