fix(@schematics/update): replace environment variables in npm/yarn rc

clydin · hansl · commit 6fce79355fb6 · 2018-11-21T20:34:36.000-08:00
diff --git a/packages/schematics/update/update/npm.ts b/packages/schematics/update/update/npm.ts
@@ -68,6 +68,11 @@ function readOptions(yarn = false): { [key: string]: string } {
     }
   }
 
+  // Substitute any environment variable references
+  for (const key in options) {
+    options[key] = options[key].replace(/\$\{([^\}]+)\}/, (_, name) => process.env[name] || '');
+  }
+
   return options;
 }
 

Original file line number	Diff line number	Diff line change
`@@ -68,6 +68,11 @@ function readOptions(yarn = false): { [key: string]: string } {`
`68`	`68`	`}`
`69`	`69`	`}`
`70`	`70`
	`71`	`+ // Substitute any environment variable references`
	`72`	`+ for (const key in options) {`
	`73`	`+ options[key] = options[key].replace(/\$\{([^\}]+)\}/, (_, name) => process.env[name] \|\| '');`
	`74`	`+ }`
	`75`	`+`
`71`	`76`	`return options;`
`72`	`77`	`}`
`73`	`78`