Merge pull request #19 from georapbox/esnext

Convert codebase to ES6

Author: georapbox

.eslintrc.json

@@ -8,22 +8,20 @@
     "es6": true
   },
   "parserOptions": {
-    "ecmaVersion": 2017
+    "ecmaVersion": "latest"
   },
   "globals": {
     "Promise": false,
     "BigInt": false
   },
   "extends": "eslint:recommended",
   "rules": {
-    // Possible Errors
     "no-empty": ["error", {
       "allowEmptyCatch": true
     }],
     "no-use-before-define": ["error", {
       "functions": false
     }],
-    // Stylistic Issues
     "indent": ["warn", 2, {
       "SwitchCase": 1,
       "ignoredNodes": ["TemplateLiteral"]
@@ -44,7 +42,9 @@
     "operator-linebreak": ["warn", "before"],
     "brace-style": ["warn"],
     "keyword-spacing": ["warn"],
-    "object-curly-spacing": ["warn", "always"],
+    "object-curly-spacing": ["warn", "never"],
+    "array-bracket-spacing": ["warn", "never"],
+    "computed-property-spacing": ["warn", "never"],
     "space-before-blocks": ["warn", "always"],
     "spaced-comment": ["warn", "always"],
     "space-before-function-paren": ["warn", {
@@ -53,7 +53,6 @@
       "asyncArrow": "always"
     }],
     "padded-blocks": ["warn", "never"],
-    // Best Practices
     "curly": ["warn"],
     "eqeqeq": ["error", "always", {
       "null": "ignore"
@@ -63,6 +62,11 @@
       "exceptions": {
         "Property": false
       }
-    }]
+    }],
+    "prefer-rest-params": ["warn"],
+    "prefer-const": ["warn"],
+    "no-var": ["warn"],
+    "no-extra-parens": ["warn", "all"],
+    "space-in-parens": ["warn", "never"]
   }
 }

.github/workflows/ci.yml

@@ -16,7 +16,7 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [12.x, 14.x]
+        node-version: [18.x]
         # See supported Node.js release schedule at https://nodejs.org/en/about/releases/
 
     steps:

.nvmrc

@@ -0,0 +1 @@
+18

README.md

@@ -1,3 +1,4 @@
 module.exports = {
+  'testEnvironment': 'jsdom',
   coverageReporters: ['json', 'lcov', 'text', 'clover', 'html']
-};
+};