Merge branch 'sachinjoseph/support-custom-environment-variables-for-install-manager' of https://github.com/microsoft/web-build-tools into sachinjoseph/support-custom-environment-variables-for-install-manager

sachinjoseph · sachinjoseph · commit ccf59b26fbd6 · 2020-03-13T16:57:44.000-07:00
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();`