chore: migrate to eslint-config-webpack (#189)

Author: alexander-akait

.editorconfig

@@ -14,3 +14,6 @@ indent_size = 2
 
 [*.md]
 trim_trailing_whitespace = false
+
+[*.snap]
+trim_trailing_whitespace = false

.eslintrc

@@ -48,6 +48,10 @@ jobs:
           cache: yarn
       - name: Install dependencies
         run: yarn --frozen-lockfile --ignore-engines
+        if: matrix.node-version == '6.x' || matrix.node-version == '8.x' || matrix.node-version == '10.x' || matrix.node-version == '12.x' || matrix.node-version == '14.x' || matrix.node-version == '16.x'
+      - name: Install dependencies
+        run: yarn --frozen-lockfile
+        if: matrix.node-version != '6.x' && matrix.node-version != '8.x' && matrix.node-version != '10.x' && matrix.node-version != '12.x' && matrix.node-version != '14.x' && matrix.node-version != '16.x'
       - name: Run tests with coverage
         run: yarn test --ci --coverage
       - uses: codecov/codecov-action@v5

.github/workflows/test.yml

@@ -1,8 +1,11 @@
+"use strict";
+
 module.exports = {
 	printWidth: 80,
 	useTabs: true,
 	tabWidth: 2,
 	trailingComma: "none",
+	arrowParens: "always",
 	overrides: [
 		{
 			files: "*.json",

.prettierrc.js

@@ -70,12 +70,12 @@ For sync hooks, `tap` is the only valid method to add a plugin. Async hooks also
 ```js
 myCar.hooks.calculateRoutes.tapPromise(
 	"GoogleMapsPlugin",
-	(source, target, routesList) => {
+	(source, target, routesList) => 
 		// return a promise
-		return google.maps.findRoute(source, target).then((route) => {
+		 google.maps.findRoute(source, target).then((route) => {
 			routesList.add(route);
-		});
-	}
+		})
+	
 );
 myCar.hooks.calculateRoutes.tapAsync(
 	"BingMapsPlugin",
@@ -106,7 +106,7 @@ class Car {
 	/**
 	 * You won't get returned value from SyncHook or AsyncParallelHook,
 	 * to do that, use SyncWaterfallHook and AsyncSeriesWaterfallHook respectively
-	 **/
+	 */
 
 	setSpeed(newSpeed) {
 		// following call returns undefined even when you returned values
@@ -117,10 +117,10 @@ class Car {
 		const routesList = new List();
 		return this.hooks.calculateRoutes
 			.promise(source, target, routesList)
-			.then((res) => {
+			.then((res) => 
 				// res is undefined for AsyncParallelHook
-				return routesList.getRoutes();
-			});
+				 routesList.getRoutes()
+			);
 	}
 
 	useNavigationSystemAsync(source, target, callback) {

README.md

@@ -0,0 +1,20 @@
+import { defineConfig } from "eslint/config";
+import config from "eslint-config-webpack";
+
+export default defineConfig([
+	{
+		extends: [config],
+		rules: {
+			"no-new-func": "off",
+			"n/prefer-node-protocol": "off"
+		}
+	},
+	{
+		languageOptions: {
+			parserOptions: {
+				ecmaVersion: 2018
+			}
+		},
+		files: ["lib/__tests__/**/*.js"]
+	}
+]);

eslint.config.mjs

@@ -49,7 +49,7 @@ class AsyncParallelBailHookCodeFactory extends HookCodeFactory {
 				code += "}\n";
 				return code;
 			},
-			onTap: (i, run, done, doneBreak) => {
+			onTap: (i, run, done, _doneBreak) => {
 				let code = "";
 				if (i > 0) {
 					code += `if(${i} >= _results.length) {\n`;
@@ -68,10 +68,10 @@ class AsyncParallelBailHookCodeFactory extends HookCodeFactory {
 
 const factory = new AsyncParallelBailHookCodeFactory();
 
-const COMPILE = function (options) {
+function COMPILE(options) {
 	factory.setup(this, options);
 	return factory.create(options);
-};
+}
 
 function AsyncParallelBailHook(args = [], name = undefined) {
 	const hook = new Hook(args, name);

lib/AsyncParallelBailHook.js

@@ -18,10 +18,10 @@ class AsyncParallelHookCodeFactory extends HookCodeFactory {
 
 const factory = new AsyncParallelHookCodeFactory();
 
-const COMPILE = function (options) {
+function COMPILE(options) {
 	factory.setup(this, options);
 	return factory.create(options);
-};
+}
 
 function AsyncParallelHook(args = [], name = undefined) {
 	const hook = new Hook(args, name);

lib/AsyncParallelHook.js

@@ -23,10 +23,10 @@ class AsyncSeriesBailHookCodeFactory extends HookCodeFactory {
 
 const factory = new AsyncSeriesBailHookCodeFactory();
 
-const COMPILE = function (options) {
+function COMPILE(options) {
 	factory.setup(this, options);
 	return factory.create(options);
-};
+}
 
 function AsyncSeriesBailHook(args = [], name = undefined) {
 	const hook = new Hook(args, name);

lib/AsyncSeriesBailHook.js

@@ -18,10 +18,10 @@ class AsyncSeriesHookCodeFactory extends HookCodeFactory {
 
 const factory = new AsyncSeriesHookCodeFactory();
 
-const COMPILE = function (options) {
+function COMPILE(options) {
 	factory.setup(this, options);
 	return factory.create(options);
-};
+}
 
 function AsyncSeriesHook(args = [], name = undefined) {
 	const hook = new Hook(args, name);