@lipemat/js-boilerplate 9.4.4 → 9.5.0

package/helpers/WebpackAssetsHash.js

@@ -51,7 +51,7 @@ class WebpackAssetsHash {
 	 */
 	storeContentHash( compilation ) {
 		for ( const asset of compilation.getAssets() ) {
-			this.assets[ asset.name ] = crypto.createHash( 'md4' )
+			this.assets[ asset.name ] = crypto.createHash( 'md5' )
 				.update( asset.source.source() )
 				.digest( 'hex' )
 				.substring( 0, 20 );

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lipemat/js-boilerplate",
-  "version": "9.4.4",
+  "version": "9.5.0",
   "description": "Dependencies and scripts for a no config JavaScript app",
   "author": "Mat Lipe",
   "license": "MIT",

package/scripts/lint.js

@@ -1,6 +1,17 @@
 const {ESLint} = require( 'eslint' );
 const packageConfig = require( '../helpers/package-config' );
 
+/**
+ * ESLint does not have a utility method for detecting if
+ * any error has occurred, nor does it set the exit code.
+ *
+ * We can use this function to determine if any error has
+ * occurred and set the exit code accordingly.
+ */
+function errorOccurred( results ) {
+	return results.some( result => result.errorCount > 0 || result.warningCount > 0 );
+}
+
 /**
  * Use the public API to run the eslint commands.
  *
@@ -29,6 +40,8 @@ const packageConfig = require( '../helpers/package-config' );
 	if ( '' === resultText ) {
 		console.log( '>> Linted JS files without errors.' );
 		console.log( '-----------------------------------' );
+	} else if ( errorOccurred( results ) ) {
+		process.exitCode = 1;
 	}
 }() ).catch( error => {
 	process.exitCode = 1;