search

blitz-js / legacy-framework

MIT License
3 stars 2 forks source link

Fix `blitz install` to work with pnpm #318

Closed flybayer closed 3 years ago

flybayer commented 3 years ago

What do you want and why?

Currently blitz install only knows about yarn or npm. So when you run blitz install tailwind in a pnpm repo, the code doesn't see yarn, so installs all the deps with npm.

This code here needs fixed: https://github.com/blitz-js/blitz/blob/canary/packages/cli/src/commands/install.ts#L245-L249

P.s. general pnpm support is just now working with https://github.com/blitz-js/blitz/pull/2673. You can test with blitz@0.40.0-danger.0a8df2b8a.10

satya-nutella commented 3 years ago

@flybayer This one's numero 4 for me. Can I take this up?

flybayer commented 3 years ago

Sure!