Update apps/rush-lib/src/cli/actions/BaseInstallAction.ts

sachinjoseph · iclanton · web-flow · commit 5599f7421b0a · 2020-03-13T13:27:45.000-07:00
Co-Authored-By: Ian Clanton-Thuon &lt;iclanton@users.noreply.github.com&gt;
diff --git a/apps/rush-lib/src/cli/actions/BaseInstallAction.ts b/apps/rush-lib/src/cli/actions/BaseInstallAction.ts
@@ -107,7 +107,7 @@ export abstract class BaseInstallAction extends BaseRushAction {
     // Because the 'defautltValue' option on the _maxInstallAttempts parameter is set,
     // it is safe to assume that the value is not null
     if (this._maxInstallAttempts.value! < 1) {
-      throw new Error(`The value of "${this._maxInstallAttempts.longName}" must be positive.`);
+      throw new Error(`The value of "${this._maxInstallAttempts.longName}" must be positive and nonzero.`);
     }
 
     const installManagerOptions: IInstallManagerOptions = this.buildInstallOptions();

Original file line number	Diff line number	Diff line change
`@@ -107,7 +107,7 @@ export abstract class BaseInstallAction extends BaseRushAction {`
`107`	`107`	`// Because the 'defautltValue' option on the _maxInstallAttempts parameter is set,`
`108`	`108`	`// it is safe to assume that the value is not null`
`109`	`109`	`if (this._maxInstallAttempts.value! < 1) {`
`110`		- throw new Error(`The value of "${this._maxInstallAttempts.longName}" must be positive.`);
	`110`	+ throw new Error(`The value of "${this._maxInstallAttempts.longName}" must be positive and nonzero.`);
`111`	`111`	`}`
`112`	`112`
`113`	`113`	`const installManagerOptions: IInstallManagerOptions = this.buildInstallOptions();`