chore: upgrade ESLint

Upgrades ESLint and also dumbs the rules down to only the built in
recommended set.

This will lose some nice to have rule configs, but we can always
re-enable them later if we feel the need.

Author: 43081j

eslint.config.js

@@ -0,0 +1,22 @@
+import eslintjs from '@eslint/js';
+
+const {configs: eslintConfigs} = eslintjs;
+
+export default [
+  {
+    ...eslintConfigs.recommended,
+    files: ['**/*.js']
+  },
+  {
+    files: ['test/**/*.js'],
+    languageOptions: {
+      globals: {
+        describe: 'readonly',
+        it: 'readonly',
+        beforeEach: 'readonly',
+        afterEach: 'readonly',
+        expect: 'readonly'
+      }
+    },
+  }
+];

index.js

@@ -1,10 +1,8 @@
 function isErrorInstance(obj) {
-  // eslint-disable-next-line prefer-reflect
   return obj instanceof Error || Object.prototype.toString.call(obj) === '[object Error]';
 }
 
 function isRegExp(obj) {
-  // eslint-disable-next-line prefer-reflect
   return Object.prototype.toString.call(obj) === '[object RegExp]';
 }
 
@@ -73,7 +71,7 @@ function compatibleMessage(thrown, errMatcher) {
   if (isRegExp(errMatcher)) {
     return errMatcher.test(comparisonString);
   } else if (typeof errMatcher === 'string') {
-    return comparisonString.indexOf(errMatcher) !== -1; // eslint-disable-line no-magic-numbers
+    return comparisonString.indexOf(errMatcher) !== -1;
   }
 
   return false;
@@ -100,7 +98,7 @@ function getConstructorName(errorLike) {
     // of the error just in case it's a poorly-constructed error. Please see chaijs/chai/issues/45 to know more.
     constructorName = errorLike.name;
     if (constructorName === '') {
-      const newConstructorName = (new errorLike().name); // eslint-disable-line new-cap
+      const newConstructorName = (new errorLike().name);
       constructorName = newConstructorName || constructorName;
     }
   }