Skip to content

Conversation

Turbo87
Copy link
Member

@Turbo87 Turbo87 commented Aug 15, 2022

https://pnpm.io/ is a different package manager for JS code, similar to yarn and npm. Its main advantage is saving a lot of disk space if a developer has multiple projects using similar dependencies, because it only stores a single copy of each dependency version on the disk, instead of one-per-project. This ultimately also leads to small speed savings.

I've been successfully using pnpm for a while on several other projects and it's been working quite well so far.

The main downside for us is that the official Heroku Node.js buildpack does not support pnpm (yet). That means we need to use an unofficial buildpack that does support it. We're using an unofficial buildpack for Rust too though, so I'm not too worried about the security implications and the pnpm buildpack appears to be maintained by a legitimate company.

@Turbo87 Turbo87 added A-frontend 🐹 C-internal 🔧 Category: Nonessential work that would make the codebase more consistent or clear labels Aug 15, 2022
@Turbo87 Turbo87 requested review from locks and pichfl August 15, 2022 17:33
@Turbo87 Turbo87 force-pushed the pnpm branch 3 times, most recently from 450a267 to 7ba83a3 Compare August 15, 2022 17:39
@bors
Copy link
Contributor

bors commented Aug 16, 2022

☔ The latest upstream changes (presumably a567f95) made this pull request unmergeable. Please resolve the merge conflicts.

@Turbo87 Turbo87 merged commit 6da9189 into rust-lang:master Aug 20, 2022
@Turbo87 Turbo87 deleted the pnpm branch August 20, 2022 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-frontend 🐹 C-internal 🔧 Category: Nonessential work that would make the codebase more consistent or clear
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants