From 1b7d0a4e4bbbc287f648dc00eb67549dbf2811b0 Mon Sep 17 00:00:00 2001 From: Patrik Oldsberg Date: Fri, 6 Mar 2020 16:23:54 +0100 Subject: [PATCH] cli: fix for error output being ignored --- packages/cli/src/commands/plugin/build.ts | 5 ++++- packages/cli/src/commands/plugin/lint.ts | 5 ++++- packages/cli/src/commands/plugin/test.ts | 5 ++++- 3 files changed, 12 insertions(+), 3 deletions(-) diff --git a/packages/cli/src/commands/plugin/build.ts b/packages/cli/src/commands/plugin/build.ts index 31c542ff27..fac78b650a 100644 --- a/packages/cli/src/commands/plugin/build.ts +++ b/packages/cli/src/commands/plugin/build.ts @@ -17,7 +17,10 @@ export default async (cmd: Command) => { } try { - spawnSync('tsc', args, { stdio: 'inherit' }); + const result = spawnSync('tsc', args, { stdio: 'inherit' }); + if (result.error) { + throw result.error; + } } catch (error) { process.stderr.write(`${chalk.red(error.message)}\n`); process.exit(1); diff --git a/packages/cli/src/commands/plugin/lint.ts b/packages/cli/src/commands/plugin/lint.ts index 133f449313..7c5be33147 100644 --- a/packages/cli/src/commands/plugin/lint.ts +++ b/packages/cli/src/commands/plugin/lint.ts @@ -5,7 +5,10 @@ export default async () => { const args = ['lint']; try { - spawnSync('web-scripts', args, { stdio: 'inherit' }); + const result = spawnSync('web-scripts', args, { stdio: 'inherit' }); + if (result.error) { + throw result.error; + } } catch (error) { process.stderr.write(`${chalk.red(error.message)}\n`); process.exit(1); diff --git a/packages/cli/src/commands/plugin/test.ts b/packages/cli/src/commands/plugin/test.ts index c881bbdfcb..6667f97055 100644 --- a/packages/cli/src/commands/plugin/test.ts +++ b/packages/cli/src/commands/plugin/test.ts @@ -10,7 +10,10 @@ export default async (cmd: Command) => { } try { - spawnSync('web-scripts', args, { stdio: 'inherit' }); + const result = spawnSync('web-scripts', args, { stdio: 'inherit' }); + if (result.error) { + throw result.error; + } } catch (error) { process.stderr.write(`${chalk.red(error.message)}\n`); process.exit(1);