@lipemat/js-boilerplate 10.4.2 → 10.5.1

package/bin/lipemat-js-boilerplate.js

@@ -18,6 +18,7 @@ const nodeArgs = scriptIndex > 0 ? args.slice( 0, scriptIndex ) : [];
 
 const TS_CONVERTED_SCRIPTS = [
 	'test',
+	'lint',
 ];
 
 switch ( script ) {

package/helpers/config.js

@@ -173,7 +173,6 @@ const getDefaultBrowsersList = () => {
  * Adjust the browserslist to include our defaults.
  */
 function adjustBrowserslist( browserRules ) {
-	browserRules.push( 'not and_uc 15.5' );
 	browserRules.push( 'not op_mini all' );
 	return browserRules;
 }

package/helpers/package-config.d.ts

@@ -0,0 +1,43 @@
+export interface PackageConfig {
+	author?: string;
+	brotliFiles: boolean;
+	certificates?: Certificates;
+	combinedJson: boolean;
+	css_folder: string;
+	dependencies: Dependencies;
+	description?: string;
+	devDependencies: Dependencies;
+	es6Modules?: string[];
+	jsPath: string;
+	license?: string;
+	name?: string;
+	packageDirectory: string;
+	packageManager?: string;
+	resolutions?: Dependencies;
+	scripts: Partial<Scripts>;
+	shortCssClasses: boolean;
+	url: string;
+	version?: string;
+	workingDirectory: string;
+}
+
+export interface Dependencies {
+	[ name: string ]: string;
+}
+
+export interface Certificates {
+	cert: string;
+	key: string;
+}
+
+export interface Scripts {
+	browserslist: string;
+	dist: string;
+	lint: string;
+	postinstall: string;
+	start: string;
+	test: string;
+}
+
+declare const packageConfig: PackageConfig;
+export default packageConfig;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lipemat/js-boilerplate",
-  "version": "10.4.2",
+  "version": "10.5.1",
   "description": "Dependencies and scripts for a no config JavaScript app",
   "author": "Mat Lipe",
   "license": "MIT",
@@ -34,6 +34,7 @@
     "lipemat-js-boilerplate": "bin/lipemat-js-boilerplate.js"
   },
   "scripts": {
+    "browserslist": "lipemat-js-boilerplate browserslist",
     "test": "lipemat-js-boilerplate test"
   },
   "dependencies": {
@@ -56,7 +57,7 @@
     "clean-css": "^5.3.0",
     "clean-webpack-plugin": "^4.0.0",
     "compression-webpack-plugin": "^10.0.0",
-    "core-js": "~3.31.1",
+    "core-js": "^3.33.1",
     "cross-spawn": "^6.0.5",
     "css-loader": "6.7.1",
     "fork-ts-checker-webpack-plugin": "^7.2.13",
@@ -94,5 +95,5 @@
     "memfs": "^3.5.3",
     "setimmediate": "^1.0.5"
   },
-  "packageManager": "yarn@3.6.1"
+  "packageManager": "yarn@3.6.4"
 }

package/scripts/{lint.js → lint.ts}

@@ -1,5 +1,17 @@
-const {ESLint} = require( 'eslint' );
-const packageConfig = require( '../helpers/package-config' );
+import {ESLint} from 'eslint';
+
+import packageConfig from '../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: ESLint.LintResult[] ): boolean {
+	return results.some( result => result.errorCount > 0 || result.warningCount > 0 );
+}
 
 /**
  * Use the public API to run the eslint commands.
@@ -10,10 +22,12 @@ const packageConfig = require( '../helpers/package-config' );
 	// 1. Create an instance with the `fix` option.
 	const eslint = new ESLint( {
 		fix: true,
+		cache: true,
+		cacheStrategy: 'content',
 	} );
 
 	// 2. Lint files. This doesn't modify target files.
-	const results = await eslint.lintFiles( [
+	const results: ESLint.LintResult[] = await eslint.lintFiles( [
 		packageConfig.workingDirectory + '/src/**/*.{js,jsx,ts,tsx}',
 	] );
 
@@ -29,6 +43,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;