Address PR feedback

Author: dmichon-msft

libraries/rush-lib/src/logic/operations/ShellOperationRunner.ts

@@ -73,7 +73,7 @@ export class ShellOperationRunner implements IOperationRunner {
         let hasWarningOrError: boolean = false;
 
         // Run the operation
-        terminal.writeLine('Invoking: ' + this._commandToRun);
+        terminal.writeLine(`Invoking: ${this._commandToRun}`);
 
         const { rushConfiguration, projectFolder } = this._rushProject;
 

libraries/rush-lib/src/logic/operations/ShellOperationRunnerPlugin.ts

@@ -99,11 +99,11 @@ export function initializeShellOperationRunner(options: {
   }
 
   if (rawCommandToRun) {
-    const { commandForHash: rawCommandForHash } = options;
+    const { commandForHash: rawCommandForHash, environment, customParameterValues } = options;
 
-    const commandToRun: string = formatCommand(rawCommandToRun, options.customParameterValues);
+    const commandToRun: string = formatCommand(rawCommandToRun, customParameterValues);
     const commandForHash: string = rawCommandForHash
-      ? formatCommand(rawCommandForHash, options.customParameterValues)
+      ? formatCommand(rawCommandForHash, customParameterValues)
       : commandToRun;
 
     return new ShellOperationRunner({
@@ -112,7 +112,7 @@ export function initializeShellOperationRunner(options: {
       displayName,
       phase,
       rushProject: project,
-      environment: options.environment
+      environment
     });
   } else {
     // Empty build script indicates a no-op, so use a no-op runner