Merge pull request #305 from thewhodidthis/nf#304--eslint

Switch to ESlint

.eslintrc.cjs

@@ -0,0 +1,140 @@
+module.exports = {
+  extends: 'eslint:recommended',
+  parserOptions: {
+    ecmaVersion: 11,
+    sourceType: 'module'
+  },
+  env: {
+    browser: true,
+    node: true,
+    es6: true
+  },
+  plugins: [],
+  rules: {
+    'accessor-pairs': 1,
+    'arrow-body-style': [2, 'as-needed'],
+    'arrow-parens': [2, 'as-needed', { requireForBlockBody: true }],
+    'arrow-spacing': 2,
+    'block-spacing': 2,
+    'brace-style': [2, 'allman', { allowSingleLine: true }],
+    'camelcase': 1,
+    'capitalized-comments': [1, 'always', { ignoreConsecutiveComments: true }],
+    'comma-dangle': 2,
+    'comma-spacing': 2,
+    'comma-style': 2,
+    'consistent-return': 1,
+    'consistent-this': 2,
+    'curly': 2,
+    'default-case': 2,
+    'dot-location': [2, 'property'],
+    'dot-notation': 1,
+    'eol-last': 2,
+    'eqeqeq': [2, 'always'],
+    'func-call-spacing': 2,
+    'func-style': [2, 'declaration', { allowArrowFunctions: true }],
+    'generator-star-spacing': 2,
+    'global-require': 2,
+    'handle-callback-err': [2, '^(err|error)$' ],
+    'indent': [2, 'tab'],
+    'key-spacing': 2,
+    'keyword-spacing': 2,
+    'line-comment-position': 2,
+    'lines-around-comment': [2, { afterLineComment: false }],
+    'max-len': [1, 100],
+    'new-cap': [2, { capIsNew: false }],
+    'new-parens': 2,
+    'no-array-constructor': 2,
+    'no-buffer-constructor': 2,
+    'no-caller': 2,
+    'no-confusing-arrow': [2, { allowParens: true }],
+    'no-console': 1,
+    'no-div-regex': 2,
+    'no-duplicate-imports': 2,
+    'no-else-return': 2,
+    'no-eval': 2,
+    'no-extend-native': 2,
+    'no-extra-bind': 2,
+    'no-extra-label': 2,
+    'no-extra-parens': [2, 'functions'],
+    'no-floating-decimal': 2,
+    'no-implied-eval': 2,
+    'no-inline-comments': 2,
+    'no-invalid-this': 2,
+    'no-iterator': 2,
+    'no-label-var': 2,
+    'no-labels': 2,
+    'no-lone-blocks': 2,
+    'no-mixed-operators': 2,
+    'no-mixed-requires': 2,
+    'no-multi-spaces': 2,
+    'no-multi-str': 2,
+    'no-multiple-empty-lines': [2, { max: 1, maxEOF: 0 }],
+    'no-new': 2,
+    'no-new-func': 2,
+    'no-new-object': 2,
+    'no-new-require': 2,
+    'no-new-wrappers': 2,
+    'no-octal-escape': 2,
+    'no-param-reassign': 1,
+    'no-path-concat': 2,
+    'no-plusplus': 2,
+    'no-proto': 2,
+    'no-restricted-properties': 2,
+    'no-return-assign': [2, 'except-parens'],
+    'no-return-await': 2,
+    'no-self-compare': 2,
+    'no-sequences': 2,
+    'no-shadow': 2,
+    'no-shadow-restricted-names': 2,
+    'no-tabs': [1, { allowIndentationTabs: true }],
+    'no-template-curly-in-string': 2,
+    'no-throw-literal': 2,
+    'no-trailing-spaces': 2,
+    'no-undef-init': 2,
+    'no-underscore-dangle': 0,
+    'no-unmodified-loop-condition': 2,
+    'no-unneeded-ternary': 2,
+    'no-unused-expressions': 2,
+    'no-use-before-define': [2, { functions: false }],
+    'no-useless-call': 2,
+    'no-useless-computed-key': 2,
+    'no-useless-constructor': 2,
+    'no-useless-rename': 2,
+    'no-useless-return': 2,
+    'no-var': 2,
+    'no-void': 2,
+    'no-whitespace-before-property': 2,
+    'no-with': 2,
+    'object-property-newline': [2, { allowMultiplePropertiesPerLine: true }],
+    'one-var': [2, 'never'],
+    'one-var-declaration-per-line': 2,
+    'operator-linebreak': [2, 'before'],
+    'padded-blocks': [2, 'never'],
+    'padding-line-between-statements': 2,
+    'prefer-const': 2,
+    'prefer-promise-reject-errors': 2,
+    'prefer-rest-params': 1,
+    'prefer-template': 2,
+    'quote-props': [2, 'consistent-as-needed'],
+    'quotes': [2, 'single'],
+    'radix': 2,
+    'require-await': 2,
+    'rest-spread-spacing': 2,
+    // 'semi': [2, 'never'],
+    // 'semi-spacing': 2,
+    'space-before-blocks': 2,
+    'space-before-function-paren': [2, { named: 'never' }],
+    'space-in-parens': 2,
+    'space-infix-ops': 2,
+    'space-unary-ops': 2,
+    'spaced-comment': 2,
+    'symbol-description': 2,
+    'template-curly-spacing': 2,
+    'template-tag-spacing': 2,
+    'unicode-bom': 2,
+    'valid-jsdoc': 2,
+    'wrap-iife': [2, 'any', { functionPrototypeMethods: true }],
+    'yield-star-spacing': [2, 'both'],
+    'yoda': 2
+  }
+}

package.json

@@ -16,7 +16,7 @@
     "build:js": "rollup -c",
     "lint": "npm run lint:js && npm run lint:css",
     "lint:css": "stylelint src/**/*.css",
-    "lint:js": "jshint rollup.config.js src/**/*.js",
+    "lint:js": "eslint rollup.config.js src/**/*.js",
     "start": "npm run build"
   },
   "babel": {
@@ -56,8 +56,8 @@
     "@rollup/plugin-babel": "^5.2.1",
     "cross-var": "^1.1.0",
     "cssnano": "^4.1.10",
+    "eslint": "^7.24.0",
     "jsdoc": "^3.6.6",
-    "jshint": "^2.12.0",
     "postcss": "^8.1.3",
     "postcss-cli": "^8.1.0",
     "postcss-preset-env": "^6.7.0",
@@ -66,9 +66,6 @@
     "stylelint-config-standard": "^20.0.0",
     "terser": "^5.3.8"
   },
-  "jshintConfig": {
-    "esversion": 8
-  },
   "postcss": {
     "plugins": {
       "postcss-preset-env": {},