Merge pull request microsoft#1744 from D4N14L/danade/fixBrokenInit

iclanton · web-flow · commit 7045be0e4548 · 2020-02-12T13:31:54.000-08:00
Move RushConfiguration access in action to unblock Init
diff --git a/apps/rush-lib/src/cli/actions/PublishAction.ts b/apps/rush-lib/src/cli/actions/PublishAction.ts
@@ -66,12 +66,6 @@ export class PublishAction extends BaseRushAction {
       'changes and publish packages, you must use the --commit flag and/or the --publish flag.',
       parser
     });
-
-    // Example: "common\temp\publish-home"
-    this._targetNpmrcPublishFolder = path.join(this.rushConfiguration.commonTempFolder, 'publish-home');
-
-    // Example: "common\temp\publish-home\.npmrc"
-    this._targetNpmrcPublishPath = path.join(this._targetNpmrcPublishFolder, '.npmrc');
   }
 
   protected onDefineParameters(): void {
@@ -211,6 +205,12 @@ export class PublishAction extends BaseRushAction {
     return Promise.resolve().then(() => {
       PolicyValidator.validatePolicy(this.rushConfiguration, false);
 
+      // Example: "common\temp\publish-home"
+      this._targetNpmrcPublishFolder = path.join(this.rushConfiguration.commonTempFolder, 'publish-home');
+
+      // Example: "common\temp\publish-home\.npmrc"
+      this._targetNpmrcPublishPath = path.join(this._targetNpmrcPublishFolder, '.npmrc');
+
       const allPackages: Map<string, RushConfigurationProject> = this.rushConfiguration.projectsByName;
 
       if (this._regenerateChangelogs.value) {
diff --git a/common/changes/@microsoft/rush/danade-fixBrokenInit_2020-02-12-02-01.json b/common/changes/@microsoft/rush/danade-fixBrokenInit_2020-02-12-02-01.json
@@ -0,0 +1,11 @@
+{
+  "changes": [
+    {
+      "packageName": "@microsoft/rush",
+      "comment": "",
+      "type": "none"
+    }
+  ],
+  "packageName": "@microsoft/rush",
+  "email": "3473356+D4N14L@users.noreply.github.com"
+}
