fix: move process exit up to the CLI (tajo#18)

talshani · Mar 30, 2021 · 9034b6a · 9034b6a
1 parent 79e2c4c
commit 9034b6a
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 8 deletions.
diff --git a/packages/ladle/lib/cli/build.js b/packages/ladle/lib/cli/build.js
@@ -59,8 +59,7 @@ const build = async (params = {}) => {
 
   debug(`Final config:\n${JSON.stringify(config, null, "  ")}`);
   process.env["SNOWPACK_PUBLIC_LADLE_THEME"] = config.addons.theme.defaultState;
-  await snowpackProd(config, configFolder);
-  process.exit(0);
+  return snowpackProd(config, configFolder);
 };
 
 export default build;
diff --git a/packages/ladle/lib/cli/cli.js b/packages/ladle/lib/cli/cli.js
@@ -41,6 +41,12 @@ program
     "additional folders where source code can be",
     (str) => str.split(","),
   )
-  .action(build);
+  .action(async (params) => {
+    const success = await build(params);
+    if (success) {
+      process.exit(0);
+    }
+    process.exit(1);
+  });
 
 program.parse(process.argv);
diff --git a/packages/ladle/lib/cli/snowpack-prod.js b/packages/ladle/lib/cli/snowpack-prod.js
@@ -40,12 +40,9 @@ const snowpackProd = async (config, configFolder) => {
     await build({ config: buildConfig, lockfile: null });
   } catch (e) {
     console.log(e);
-    process.exit(1);
+    return false;
   }
-  if (successfulExit) {
-    process.exit(0);
-  }
-  process.exit(1);
+  return successfulExit;
 };
 
 export default snowpackProd;