tinyglobby icon indicating copy to clipboard operation
tinyglobby copied to clipboard

chore: set onlyBuiltDependencies for pnpm

Open benmccann opened this issue 7 months ago • 3 comments

You have to tell the latest version of pnpm that it's allowed to run the post-install scripts for these packages or it will complain when you do pnpm install

benmccann avatar Apr 19 '25 16:04 benmccann

Open in StackBlitz

npm i https://pkg.pr.new/tinyglobby@106

commit: 41dc943

pkg-pr-new[bot] avatar Apr 19 '25 16:04 pkg-pr-new[bot]

do we really need it though? biome is definitely not needed as it only checks if it's being installed in a supported platform (in fact the script won't be there for biome 2.0). wondering about esbuild

SuperchupuDev avatar Apr 19 '25 16:04 SuperchupuDev

It's going to nag if it's not there. Regardless of whether anything is broken without it, I'd add it to avoid the warning

benmccann avatar Apr 19 '25 17:04 benmccann

since switching to tsdown, according to ci it looks like the only package with a postinstall script is biome, and that script was recently deleted for the upcoming biome 2.0 release. the warning should no longer be there once biome 2.0 releases

SuperchupuDev avatar Jun 14 '25 15:06 SuperchupuDev

after upgrading to biome 2.0, tinyglobby now doesn't use a single package with postinstall scripts, making the warning no longer show up. as such this is probably not needed anymore

SuperchupuDev avatar Jun 18 '25 11:06 SuperchupuDev