Remove extraneous 'throw'

Geoff Vandegrift · Geoff Vandegrift · commit cd027538a1f2 · 2016-10-25T14:42:48.000-05:00
diff --git a/lib/index.js b/lib/index.js
@@ -62,7 +62,7 @@ class WebpackShellPlugin {
       }
       if (this.options.onBuildStart.length) {
         console.log('Executing pre-build scripts');
-        throw this.options.onBuildStart.forEach(script => {
+        this.options.onBuildStart.forEach(script => {
           if (this.options.throwOnExecError) {
             spreadStdoutAndStdErr(exec(script, putsThrow));
           } else {
diff --git a/src/webpack-shell-plugin.js b/src/webpack-shell-plugin.js
@@ -60,7 +60,7 @@ export default class WebpackShellPlugin {
       }
       if (this.options.onBuildStart.length) {
         console.log('Executing pre-build scripts');
-        throw this.options.onBuildStart.forEach((script) => {
+        this.options.onBuildStart.forEach((script) => {
           if (this.options.throwOnExecError) {
             spreadStdoutAndStdErr(exec(script, putsThrow));
           }

Original file line number	Diff line number	Diff line change
`@@ -62,7 +62,7 @@ class WebpackShellPlugin {`
`62`	`62`	`}`
`63`	`63`	`if (this.options.onBuildStart.length) {`
`64`	`64`	`console.log('Executing pre-build scripts');`
`65`		`- throw this.options.onBuildStart.forEach(script => {`
	`65`	`+ this.options.onBuildStart.forEach(script => {`
`66`	`66`	`if (this.options.throwOnExecError) {`
`67`	`67`	`spreadStdoutAndStdErr(exec(script, putsThrow));`
`68`	`68`	`} else {`
Original file line number	Diff line number	Diff line change
`@@ -60,7 +60,7 @@ export default class WebpackShellPlugin {`
`60`	`60`	`}`
`61`	`61`	`if (this.options.onBuildStart.length) {`
`62`	`62`	`console.log('Executing pre-build scripts');`
`63`		`- throw this.options.onBuildStart.forEach((script) => {`
	`63`	`+ this.options.onBuildStart.forEach((script) => {`
`64`	`64`	`if (this.options.throwOnExecError) {`
`65`	`65`	`spreadStdoutAndStdErr(exec(script, putsThrow));`
`66`	`66`	`}`