Merge pull request #3495 from dbartholomae/set_default_max-install-at…

…tempts_to_1 [rush-lib] Set default max install attempts to 1
microsoft · Jun 27, 2022 · 5fd4f28 · 5fd4f28
2 parents 4b759f4 + f045fa2
commit 5fd4f28
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 3 deletions.
diff --git a/common/changes/@microsoft/rush/set_default_max-install-attempts_to_1_2022-06-26-12-16.json b/common/changes/@microsoft/rush/set_default_max-install-attempts_to_1_2022-06-26-12-16.json
@@ -0,0 +1,10 @@
+{
+  "changes": [
+    {
+      "packageName": "@microsoft/rush",
+      "comment": "Reduce default maxInstallAttempts to 1",
+      "type": "none"
+    }
+  ],
+  "packageName": "@microsoft/rush"
+}
diff --git a/libraries/rush-lib/src/cli/test/__snapshots__/CommandLineHelp.test.ts.snap b/libraries/rush-lib/src/cli/test/__snapshots__/CommandLineHelp.test.ts.snap
@@ -589,7 +589,7 @@ Optional arguments:
                         a file when using this command.
   --max-install-attempts NUMBER
                         Overrides the default maximum number of install 
-                        attempts. The default value is 3.
+                        attempts. The default value is 1.
   --ignore-hooks        Skips execution of the \\"eventHooks\\" scripts defined 
                         in rush.json. Make sure you know what you are 
                         skipping.
@@ -1135,7 +1135,7 @@ Optional arguments:
                         a file when using this command.
   --max-install-attempts NUMBER
                         Overrides the default maximum number of install 
-                        attempts. The default value is 3.
+                        attempts. The default value is 1.
   --ignore-hooks        Skips execution of the \\"eventHooks\\" scripts defined 
                         in rush.json. Make sure you know what you are 
                         skipping.

diff --git a/libraries/rush-lib/src/logic/RushConstants.ts b/libraries/rush-lib/src/logic/RushConstants.ts
@@ -73,7 +73,7 @@ export class RushConstants {
   /**
    * Number of installation attempts
    */
-  public static readonly defaultMaxInstallAttempts: number = 3;
+  public static readonly defaultMaxInstallAttempts: number = 1;
 
   /**
    * The filename ("pnpm-lock.yaml") used to store an installation plan for the PNPM package manger