-
Notifications
You must be signed in to change notification settings - Fork 33
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
Add pnpm support? #113
Comments
Hey @borekb and thanks for opening this issue! I've looked into this use case and added support for both yarn-3 with on qnm version Please let me know if it works for you |
Wow that was fast! I tried in our repo which uses Yarn workspaces and it failed after about 1 minute with this error:
I guess Yarn workspaces might be the problem. I don't really have a non-workspaces project I could test right now but still appreciate your super-fast reaction! |
Hey @borekb, thanks for testing it! Actually, qnm supports yarn workspaces, I'm not sure what is the issue 🤷♂️ |
It's an internal repo, unfortunately, and I can't promise that I'll remember to create a public repro 😄. But you're awesome for adding pnpm support so quickly! |
The pnpm support seems to only recognize the top-level dependencies from my testing of running |
@beeequeue can you create a repro and describe what do you expect to see in oppose to what qnm shows today? I believe it could help shape the output according to what's relevant for the use case |
https://github.com/BeeeQueue/qnm-repro If you install using npm it shows 700~ deps, and if you remove |
Thanks @beeequeue. You are right. While I made qnm not crash on pnpm repositories, what it does is just list the top-level modules within the pnpm works differently from npm/yarn in a way that it'll create links for every package, and it will place its own dependencies as its siblings Today, qnm's algorithm is to run on a workspace and analyze it from top to bottom, it doesn't try to go "up" in the directory structure like Node's resolution algorithm does. This is the reason you see only 8 dependencies when running qnm today. qnm was created because, for years, package managers compromised on correctness for install size by hoisting packages, which created many problems that needed to be debugged. As I'm not a Once I'll understand the need, it would be possible to implement a solution for it. |
This looks like a nice tool but it doesn't seem to work with pnpm installations (we, specifically, use Yarn 3 & its pnpm linker but I guess using the pnpm CLI itself would have the same problem).
The output is just this:
The text was updated successfully, but these errors were encountered: