FIX: add exit 1 when commands fail (#1198)

Co-authored-by: leydev <[email protected]>

src/modules/apps/install.ts

@@ -53,6 +53,8 @@ function handleAccountNotSponsoredByVendorError(app: string) {
 }
 
 const prepareInstall = async (appsList: string[], force: boolean): Promise<void> => {
+  let exitWithError = false;
+
   for (const app of appsList) {
     ManifestValidator.validateApp(app)
     try {
@@ -79,6 +81,7 @@ const prepareInstall = async (appsList: string[], force: boolean): Promise<void>
       }
       log.info(`Installed app ${chalk.green(app)} successfully`)
     } catch (e) {
+      exitWithError = true;
       if (isNotFoundError(e)) {
         log.warn(
           `Billing app not found in current workspace. Please install it with ${chalk.green(
@@ -107,6 +110,8 @@ const prepareInstall = async (appsList: string[], force: boolean): Promise<void>
       log.warn(`The following app was not installed: ${app}`)
     }
   }
+
+  if (exitWithError) process.exit(1);
 }
 
 export default async (optionalApps: string[], options) => {

src/modules/apps/publish.ts

@@ -111,7 +111,8 @@ const publisher = (workspace = 'master') => {
         )
       }
     } catch (e) {
-      log.error(`Failed to publish ${appId}`)
+      log.error(`Failed to publish ${appId}`);
+      process.exit(1);
     }
 
     await returnToPreviousAccount({ previousAccount, previousWorkspace })

src/modules/apps/unlink.ts

@@ -22,6 +22,7 @@ Make sure you typed the right app vendor, name and version.`)
         log.error(e.response.data.message)
       }
     }
+    process.exit(1);
   }
 }
 
@@ -40,6 +41,7 @@ const unlinkAllApps = async (): Promise<void> => {
     if (e?.response?.data?.message) {
       log.error(e.response.data.message)
     }
+    process.exit(1);
   }
 }