fix: add quotes to file replacement switch (#54)

JakeAi · Jake Ihasz · web-flow · commit 25513fc78688 · 2022-06-15T10:19:49.000-07:00
When more than 1 file replacement is present, the comma would break the command.

Co-authored-by: Jake Ihasz &lt;JakeIhasz@minnich-mfg.com&gt;
diff --git a/packages/nx/src/utils/executors.ts b/packages/nx/src/utils/executors.ts
@@ -205,7 +205,7 @@ export function commonExecutor(options: BuildExecutorSchema | TestExecutorSchema
 
         if (nsCliFileReplacements.length) {
           // console.log('nsCliFileReplacements:', nsCliFileReplacements);
-          nsOptions.push(`--env.replace=${nsCliFileReplacements.join(',')}`);
+          nsOptions.push(`--env.replace="${nsCliFileReplacements.join(',')}"`);
         }
         // always add --force (unless explicity set to false) for now since within Nx we use @nativescript/webpack at root only and the {N} cli shows a blocking error if not within the app
         if (options?.force !== false) {

Original file line number	Diff line number	Diff line change
`@@ -205,7 +205,7 @@ export function commonExecutor(options: BuildExecutorSchema \| TestExecutorSchema`
`205`	`205`
`206`	`206`	`if (nsCliFileReplacements.length) {`
`207`	`207`	`// console.log('nsCliFileReplacements:', nsCliFileReplacements);`
`208`		- nsOptions.push(`--env.replace=${nsCliFileReplacements.join(',')}`);
	`208`	+ nsOptions.push(`--env.replace="${nsCliFileReplacements.join(',')}"`);
`209`	`209`	`}`
`210`	`210`	`// always add --force (unless explicity set to false) for now since within Nx we use @nativescript/webpack at root only and the {N} cli shows a blocking error if not within the app`
`211`	`211`	`if (options?.force !== false) {`