Fix shell

src/run/runners/cli/benchmark/spawn.js

@@ -5,7 +5,6 @@ export const spawnProcess = async function(
   command,
   { variation, shell, stdio },
 ) {
-  const shellA = String(shell) === 'true'
-  const execaFunc = shellA ? execa : execa.command
-  await execaFunc(command, { stdio, shell: shellA, preferLocal: true })
+  const execaFunc = shell ? execa : execa.command
+  await execaFunc(command, { stdio, shell, preferLocal: true })
 }

src/run/runners/cli/load/normalize.js

@@ -10,12 +10,10 @@ export const normalizeTasks = function({
   const tasksA = Object.entries(tasks).map(([taskId, task]) =>
     normalizeTask({ taskId, task, taskPath, variables }),
   )
-  const shellA = applyTemplate(shell, variables)
+  const shellA = getShell({ shell, variables })
   return { tasks: tasksA, variations, shell: shellA }
 }
 
-const DEFAULT_SHELL = true
-
 const normalizeTask = function({
   taskId,
   task: { title, variations, main, before, after },
@@ -42,3 +40,15 @@ const applyTaskTemplates = function({ title, variations, variables }) {
   )
   return { taskTitle, variationsIds }
 }
+
+// `shell` can contain variables, i.e. can be a string 'true' or 'false'
+const getShell = function({ shell = DEFAULT_SHELL, variables }) {
+  if (typeof shell === 'boolean') {
+    return shell
+  }
+
+  const shellA = applyTemplate(shell, variables)
+  return shellA === 'true'
+}
+
+const DEFAULT_SHELL = true