fix: pnpm does not want save flag on remove

farfromrefug · farfromrefug · commit 8d6b32f0e096 · 2020-01-23T09:12:23.000+01:00
diff --git a/lib/pnpm-package-manager.ts b/lib/pnpm-package-manager.ts
@@ -51,6 +51,8 @@ export class PnpmPackageManager extends BasePackageManager {
 
 	@exported("pnpm")
 	public uninstall(packageName: string, config?: IDictionary<string | boolean>, cwd?: string): Promise<string> {
+		// pnpm does not want save option in remove. It saves it by default
+		delete config['save'];
 		const flags = this.getFlagsString(config, false);
 		return this.$childProcess.exec(`pnpm remove ${packageName} ${flags}`, { cwd });
 	}

Original file line number	Diff line number	Diff line change
`@@ -51,6 +51,8 @@ export class PnpmPackageManager extends BasePackageManager {`
`51`	`51`
`52`	`52`	`@exported("pnpm")`
`53`	`53`	`public uninstall(packageName: string, config?: IDictionary<string \| boolean>, cwd?: string): Promise<string> {`
	`54`	`+ // pnpm does not want save option in remove. It saves it by default`
	`55`	`+ delete config['save'];`
`54`	`56`	`const flags = this.getFlagsString(config, false);`
`55`	`57`	return this.$childProcess.exec(`pnpm remove ${packageName} ${flags}`, { cwd });
`56`	`58`	`}`