Merge updates from project template

Including updated ESLint version and config.

Fixes: #37
Fixes: #40

Signed-off-by: Kevin Locke <[email protected]>

Author: kevinoid

.eslintrc.json

@@ -1,98 +1,8 @@
 // ESLint configuration <https://eslint.org/docs/user-guide/configuring>
 {
-  // Based on Airbnb with changes to match Node core and my prefs.
-  "extends": "airbnb-base",
-
-  "parserOptions": {
-    "sourceType": "script"
-  },
+  "extends": "@kevinoid/eslint-config/node.js",
 
   "rules": {
-    //// Possible Errors <https://eslint.org/docs/rules/#possible-errors>
-    // disallow trailing commas in array and object literals
-    "comma-dangle": ["error", "never"],
-
-    //// Best Practices <https://eslint.org/docs/rules/#best-practices>
-    // require curly braces around all control statements, not just multi-line.
-    "curly": ["error", "all"],
-    // disallow unnecessary parentheses
-    "no-extra-parens": ["error", "all", {
-      "conditionalAssign": false,
-      "enforceForArrowConditionals": false,
-      "nestedBinaryExpressions": false,
-      "returnAssign": false
-    }],
-    // allow multiple spaces only before EOL (for inline comment alignment)
-    "no-multi-spaces": ["error", { "ignoreEOLComments": true }],
-    // allow reassignment of function parameters
-    // simplicity for optional arguments outweighs the perf advantage, usually
-    "no-param-reassign": "off",
-    // disallow unnecessary use of Function.prototype.{apply,call}
-    "no-useless-call": "error",
-    // Disallow unnecessary catch clauses
-    "no-useless-catch": "error",
-
-    //// Strict Mode <https://eslint.org/docs/rules/#strict-mode>
-    // require 'use strict' in global scope
-    "strict": ["error", "global"],
-
-    //// Variables <https://eslint.org/docs/rules/#variables>
-    // disallow shadowing of variables (including builtins and hoisted funcs)
-    "no-shadow": ["error", { "builtinGlobals": true, "hoist": "all" }],
-    // disallow declaration of variables that are not used in the code
-    // allow unused args for documentation, future use, and correct .length
-    "no-unused-vars": ["error", {"args": "none"}],
-    // disallow referencing variables before declaring them, excluding functions
-    // since it is required for mutually-recursive functions
-    "no-use-before-define": ["error", "nofunc"],
-
-    //// Stylistic Issues <https://eslint.org/docs/rules/#stylistic-issues>
-    // require camel case names (even for properties and destructuring)
-    "camelcase": ["error", {
-      "ignoreDestructuring": false
-    }],
-    // allow unnamed functions
-    "func-names": "off",
-    // don't enforce consistent linebreak style
-    // Allow developers to develop with native EOL.  VCS manages committed style.
-    "linebreak-style": "off",
-    // enforce a maximum line length
-    // reduce from 100 to 80 chars (conventional terminal width)
-    // ignore lines which consist of a single string, URL, or RegExp literal,
-    // possibly prefixed with comment opener or suffixed with ";".
-    // (Not ESLint ignore props which ignores any lines which contain these.)
-    "max-len": ["error", 80, 2, {
-      "ignorePattern": "^\\s*((/?\\*|/[/])\\s*)?('[^'\\\\]*(\\\\.[^'\\\\]*)*'|\"[^\"\\\\]*(\\\\.[^\"\\\\]*)*\"|/[^/\\\\]*(\\\\.[^/\\\\]*)*/[gimuy]*|[^:/?#\\s]+:/[/]\\S+);?$"
-    }],
-    // allow nested ternary expressions
-    // if they are indented one-expression-per-line, they are clear enough for me
-    "no-nested-ternary": "off",
-    // disallow process.exit()
-    // this is disabled locally only when require.main === module
-    "no-process-exit": "error",
-    // disallow extra spaces in object literals
-    "object-curly-spacing": ["error", "never"],
-    // allow multiple variable declarations per block/function and multiple
-    // declarators per declaration
-    "one-var": "off",
-    // require initialized variables to be declared on separate lines
-    "one-var-declaration-per-line": ["error", "initializations"],
-    // Requires operator at the beginning of the line in multiline statements
-    // Airbnb prevents breaks around =, suggesting ().  I don't see the advantage.
-    // Break after = looks better to me, so first assigned operand is farther left.
-    "operator-linebreak": ["error", "before", { "overrides": { "=": "after" } }],
-    // space before function parens only for async arrow (as Node core does)
-    "space-before-function-paren": ["error", {
-      "anonymous": "never",
-      "named": "never",
-      "asyncArrow": "always"
-    }],
-
-    //// ECMAScript 6 Rules <https://eslint.org/docs/rules/#ecmascript-6>
-    // require parentheses around arrow function arguments (as Node core does)
-    "arrow-parens": ["error", "always"],
-
-    // eslint-plugin-import
     // Allow requiring devDependencies for build and test
     "import/no-extraneous-dependencies": ["error", {
       "devDependencies": [