chartdb icon indicating copy to clipboard operation
chartdb copied to clipboard

chore(readme): add note to support PNPM

Open ngdangtu-vn opened this issue 9 months ago • 3 comments

Fix #575

ngdangtu-vn avatar Feb 11 '25 05:02 ngdangtu-vn

Are you sure this isn't because of hoisting? Try creating a .npmrc in the root of your project with this inside of it:

node-linker=hoisted

And then run pnpm install again

RWOverdijk avatar Feb 11 '25 08:02 RWOverdijk

Thanks, you are right. Actually it's the first time I see this option in npmrc. So do you which do you prefer:

  1. add note about hoisting in README
  2. add .npmrc

ngdangtu-vn avatar Feb 11 '25 09:02 ngdangtu-vn

I would personally opt for the note in the readme. Hoisting sucks, but it is unfortunately needed because not every tool (or every OS for that matter) deals with links well yet. Even though it's 2025 😄

RWOverdijk avatar Feb 11 '25 11:02 RWOverdijk