Skip to content

Commit 4447a77

Browse files
committed
Remove --dev flag to install both dependencies and devDependencies
The --dev flag installs ONLY devDependencies, removing production deps. We want both, so use regular install which installs everything from package.json when PNPM_INSTALL_ENV unsets CI and NODE_ENV.
1 parent 3a12344 commit 4447a77

1 file changed

Lines changed: 5 additions & 9 deletions

File tree

scripts/install-npm-packages.mjs

Lines changed: 5 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -541,16 +541,12 @@ async function installPackage(packageInfo) {
541541
devDependencies: devDeps,
542542
})
543543

544-
// Install devDependencies (test runners, etc.).
544+
// Install devDependencies (test runners, etc.) along with dependencies.
545545
writeProgress('👷')
546-
await runCommand(
547-
'pnpm',
548-
['install', '--dev', ...PNPM_HOISTED_INSTALL_FLAGS],
549-
{
550-
cwd: packageTempDir,
551-
env: { ...process.env, ...PNPM_INSTALL_ENV },
552-
},
553-
)
546+
await runCommand('pnpm', ['install', ...PNPM_HOISTED_INSTALL_FLAGS], {
547+
cwd: packageTempDir,
548+
env: { ...process.env, ...PNPM_INSTALL_ENV },
549+
})
554550
}
555551

556552
// Apply Socket overrides selectively.

0 commit comments

Comments
 (0)