Skip to content

Commit ca6fb68

Browse files
authored
fix: remove --legacy-peer-deps flag on install (#5839)
1 parent c5620bd commit ca6fb68

File tree

2 files changed

+2
-5
lines changed

2 files changed

+2
-5
lines changed

lib/bun-package-manager.ts

+1-4
Original file line numberDiff line numberDiff line change
@@ -47,10 +47,7 @@ export class BunPackageManager extends BasePackageManager {
4747
const jsonContentBefore = this.$fs.readJson(packageJsonPath);
4848

4949
const flags = this.getFlagsString(config, true);
50-
// TODO: Confirm desired behavior. The npm version uses --legacy-peer-deps
51-
// by default, we could use `--no-peer` for Bun if similar is needed; the
52-
// pnpm version uses `--shamefully-hoist`, but Bun has no similar flag.
53-
let params = ["install", "--legacy-peer-deps"];
50+
let params = ["install"];
5451
const isInstallingAllDependencies = packageName === pathToSave;
5552
if (!isInstallingAllDependencies) {
5653
params.push(packageName);

lib/node-package-manager.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ export class NodePackageManager extends BasePackageManager {
4747
const jsonContentBefore = this.$fs.readJson(packageJsonPath);
4848

4949
const flags = this.getFlagsString(config, true);
50-
let params = ["install", "--legacy-peer-deps"];
50+
let params = ["install"];
5151
const isInstallingAllDependencies = packageName === pathToSave;
5252
if (!isInstallingAllDependencies) {
5353
params.push(packageName);

0 commit comments

Comments
 (0)