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

Consider supporting pnpm #67

Closed
bojanraic opened this issue Feb 13, 2020 · 4 comments · Fixed by #240
Closed

Consider supporting pnpm #67

bojanraic opened this issue Feb 13, 2020 · 4 comments · Fixed by #240

Comments

@bojanraic
Copy link

pnpm is a fast, disk space efficient package manager relying on hard links and symlinks to save only one copy of a module/version regardless of how many projects you have using the same version of a module or library. Read more about it at https://pnpm.js.org/

As someone who experiments a lot, I have dozens of projects using same versions of libs/modules and the disk space used across all node_modules directories for all those projects adds up fast.

It would be nice to add support for pnpm through a config flag so that developers could choose to make use of it in local builds.
Thanks!

@deepy
Copy link
Member

deepy commented Feb 13, 2020

You can definitely support this by creating NodeTasks.
I skimmed the page and I suspect it might not work as a drop-in replacement with our inputs/outputs, but I might be wrong.

@bojanraic
Copy link
Author

Thanks for the quick reply. Gee, I haven't thought of that! In my simple case, I ended up using an NpxTask, but you're absolutely right, at least for now my basic use case is fulfilled.
I'll leave the issue open if you prefer to do more investigation on drop-in replacement compatibility, otherwise, please feel free to close it.
Once again, thanks for the suggestion!

@deepy
Copy link
Member

deepy commented Jun 26, 2022

Native support for pnpm is available in 3.4.0

@deepy
Copy link
Member

deepy commented Jun 26, 2022

cc @langrp @DreierF

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