diff --git a/index.js b/index.js index 233f555..ca4d41d 100644 --- a/index.js +++ b/index.js @@ -17,7 +17,6 @@ module.exports = { 'valid-typeof': ['error', { requireStringLiterals: true }], // Best Practices https://eslint.org/docs/rules/#best-practices - 'accessor-pairs': 'off', 'array-callback-return': ['error', { allowImplicit: false }], 'block-scoped-var': 'error', 'class-methods-use-this': 'error', @@ -50,7 +49,6 @@ module.exports = { 'no-implied-eval': 'error', 'no-invalid-this': 'error', 'no-iterator': 'error', - 'no-labels': 'off', 'no-lone-blocks': 'error', 'no-loop-func': 'error', 'no-magic-numbers': ['error', { enforceConst: true, ignoreArrayIndexes: true, detectObjects: false }], @@ -92,7 +90,6 @@ module.exports = { // Variables https://eslint.org/docs/rules/#variables 'init-declarations': ['error', 'always'], 'no-label-var': 'error', - 'no-restricted-globals': 'off', 'no-shadow': ['error', { builtinGlobals: true, hoist: 'all' }], 'no-undef-init': 'error', 'no-undefined': 'error', @@ -108,7 +105,6 @@ module.exports = { 'no-path-concat': 'error', 'no-process-env': 'warn', 'no-process-exit': 'warn', - 'no-restricted-modules': 'off', 'no-sync': ['error', { allowAtRootLevel: true }], // Stylistic Issues https://eslint.org/docs/rules/#stylistic-issues @@ -118,7 +114,6 @@ module.exports = { 'block-spacing': ['error', 'always'], 'brace-style': ['error', 'stroustrup', { allowSingleLine: true }], camelcase: 'error', - 'capitalized-comments': 'off', 'comma-dangle': ['error', 'never'], 'comma-spacing': 'error', 'comma-style': 'error', @@ -131,8 +126,6 @@ module.exports = { 'func-style': ['error', 'declaration', { allowArrowFunctions: false }], 'function-call-argument-newline': ['error', 'consistent'], 'function-paren-newline': ['error', 'multiline-arguments'], - 'id-blacklist': 'off', - 'id-length': 'off', 'id-match': ['error', '^([nsoa]([A-Z][a-z]+)+|[a-z])$'], 'implicit-arrow-linebreak': ['error', 'beside'], indent: ['error', 2], @@ -173,17 +166,24 @@ module.exports = { 'always', { exceptAftersingleLine: true } ], - 'max-depth': ['error', 4], - 'max-len': ['error', { code: 80, tabWidth: 4 }], + 'max-depth': ['warn', 4], + 'max-len': ['warn', { code: 80, tabWidth: 4 }], 'max-lines': [ - 'error', + 'warn', { max: 300, skipBlankLines: true, ignoreComments: true } ], - 'max-lines-per-function': null, // https://eslint.org/docs/rules/max-lines-per-function // enforce a maximum number of line of code in a function + 'max-lines-per-function': [ + 'warn', + { + max: 50, + skipBlankLines: true, + skipComments: true, + IIFEs: true + }], 'max-nested-callbacks': null, // https://eslint.org/docs/rules/max-nested-callbacks // enforce a maximum depth that callbacks can be nested 'max-params': null, // https://eslint.org/docs/rules/max-params // enforce a maximum number of parameters in function definitions 'max-statements': null, // https://eslint.org/docs/rules/max-statements // enforce a maximum number of statements allowed in function blocks