Merge pull request #677 from Financial-Times/split-config-loading

refactor: split out task running and config loading
Financial-Times · Oct 1, 2024 · c46f3a3 · c46f3a3
2 parents 53a4604 + 5355541
commit c46f3a3
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/core/cli/src/tasks.ts b/core/cli/src/tasks.ts
@@ -56,9 +56,7 @@ const loadTasks = async (
   return reduceValidated(taskResults)
 }
 
-export async function runTasks(logger: Logger, commands: string[], files?: string[]): Promise<void> {
-  const config = await loadConfig(logger)
-
+export async function runTasksFromConfig(logger: Logger, config: ValidConfig, commands: string[], files?: string[]): Promise<void> {
   for (const pluginOptions of Object.values(config.pluginOptions)) {
     if (pluginOptions.forPlugin) {
       setOptions(pluginOptions.forPlugin.id as OptionKey, pluginOptions.options)
@@ -131,3 +129,9 @@ ${error.details}`
     }
   }
 }
+
+export async function runTasks(logger: Logger, commands: string[], files?: string[]): Promise<void> {
+  const config = await loadConfig(logger)
+
+  return runTasksFromConfig(logger, config, commands, files)
+}