Build Tooling: Reinstate error handling to build script

WordPress · May 25, 2019 · c4f908b · c4f908b
1 parent 6663433
commit c4f908b
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 4 deletions.
diff --git a/bin/packages/build-worker.js b/bin/packages/build-worker.js
@@ -145,9 +145,15 @@ const BUILD_TASK_BY_EXTENSION = {
 module.exports = async ( file, callback ) => {
 	const extension = path.extname( file );
 	const task = BUILD_TASK_BY_EXTENSION[ extension ];
-	if ( task ) {
-		await task( file );
+
+	if ( ! task ) {
+		return;
 	}
 
-	callback();
+	try {
+		await task( file );
+		callback();
+	} catch ( error ) {
+		callback( error );
+	}
 };
diff --git a/bin/packages/build.js b/bin/packages/build.js
@@ -1,3 +1,5 @@
+/* eslint-disable no-console */
+
 /**
  * External dependencies
  */
@@ -65,9 +67,13 @@ let ended = false,
 	complete = 0;
 
 stream
-	.on( 'data', ( file ) => worker( file, () => {
+	.on( 'data', ( file ) => worker( file, ( error ) => {
 		onFileComplete();
 
+		if ( error ) {
+			console.log( error );
+		}
+
 		if ( ended && ++complete === files.length ) {
 			workerFarm.end( worker );
 		}