-
-
Notifications
You must be signed in to change notification settings - Fork 14
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
ERROR: LRUCache is not a constructor #122
Comments
What version of node and npm are you running? |
I use pnpm 8 and Node 20 |
Unfortunately, I'm not seeing this behavior on my end, even when running with pnpm. I'll need more info and a reproducible case that you can share before I can do anything about this.
would be a good place to start. |
You should be able to reproduce the error here (bump npm-run-all2 version before testing): I am on Linux, Node v20.8.1, pnpm 8.9.2 |
Thanks, I see the issue now. Your pnpm override: https://github.com/aminya/patha/blob/master/package.json#L151C7-L151C28 Is causing a transitive of npm-run-all2 to resolve to an incompatible version (hosted-git-info wants LRUCache 10, but is getting 7). Remove the override and the tool continues to work. Its unfortunate that modules break their internal APIs all the time as these issue will always follow in unexpected ways though the use of override here is also part of the problem. This is just part of the cost of keeping up to to date with dependencies. Noting this as a point of future simplification for this tool but I dont have any bandwidth to fix now. I wish devDependencies didn't merge with dependencies this way but thats just the way the tools are. |
When I use
npm-run-all2@6.1.1
, I get this error when I try to userun-p
:The latest version that works is
6.0.5
The text was updated successfully, but these errors were encountered: