@gitlab/eslint-plugin 21.2.0 → 21.3.0

package/eslint9/lib/configs/base/best-practices.js

@@ -1,447 +0,0 @@
-module.exports = {
-  parserOptions: {
-    ecmaVersion: 2018,
-    sourceType: 'module',
-  },
-  rules: {
-    // enforces getter/setter pairs in objects
-    // https://eslint.org/docs/rules/accessor-pairs
-    'accessor-pairs': 'off',
-
-    // enforces return statements in callbacks of array's methods
-    // https://eslint.org/docs/rules/array-callback-return
-    'array-callback-return': ['error', { allowImplicit: true }],
-
-    // treat var statements as if they were block scoped
-    // https://eslint.org/docs/rules/block-scoped-var
-    'block-scoped-var': 'error',
-
-    // specify the maximum cyclomatic complexity allowed in a program
-    // https://eslint.org/docs/rules/complexity
-    complexity: ['off', 20],
-
-    // enforce that class methods use "this"
-    // https://eslint.org/docs/rules/class-methods-use-this
-    'class-methods-use-this': [
-      'error',
-      {
-        exceptMethods: [],
-      },
-    ],
-
-    // require return statements to either always or never specify values
-    // https://eslint.org/docs/rules/consistent-return
-    'consistent-return': 'error',
-
-    // specify curly brace conventions for all control statements
-    // https://eslint.org/docs/rules/curly
-    curly: ['error', 'multi-line'], // multiline
-
-    // require default case in switch statements
-    // https://eslint.org/docs/rules/default-case
-    'default-case': ['error', { commentPattern: '^no default$' }],
-
-    // Enforce default clauses in switch statements to be last
-    // https://eslint.org/docs/rules/default-case-last
-    'default-case-last': 'error',
-
-    // https://eslint.org/docs/rules/default-param-last
-    'default-param-last': 'error',
-
-    // encourages use of dot notation whenever possible
-    // https://eslint.org/docs/rules/dot-notation
-    'dot-notation': ['error', { allowKeywords: true }],
-
-    // enforces consistent newlines before or after dots
-    // https://eslint.org/docs/rules/dot-location
-    'dot-location': ['error', 'property'],
-
-    // require the use of === and !==
-    // https://eslint.org/docs/rules/eqeqeq
-    eqeqeq: ['error', 'always', { null: 'ignore' }],
-
-    // Require grouped accessor pairs in object literals and classes
-    // https://eslint.org/docs/rules/grouped-accessor-pairs
-    'grouped-accessor-pairs': 'error',
-
-    // make sure for-in loops have an if statement
-    // https://eslint.org/docs/rules/guard-for-in
-    'guard-for-in': 'error',
-
-    // enforce a maximum number of classes per file
-    // https://eslint.org/docs/rules/max-classes-per-file
-    'max-classes-per-file': ['error', 1],
-
-    // disallow the use of alert, confirm, and prompt
-    // https://eslint.org/docs/rules/no-alert
-    'no-alert': 'warn',
-
-    // disallow use of arguments.caller or arguments.callee
-    // https://eslint.org/docs/rules/no-caller
-    'no-caller': 'error',
-
-    // disallow lexical declarations in case/default clauses
-    // https://eslint.org/docs/rules/no-case-declarations
-    'no-case-declarations': 'error',
-
-    // Disallow returning value in constructor
-    // https://eslint.org/docs/rules/no-constructor-return
-    'no-constructor-return': 'error',
-
-    // disallow division operators explicitly at beginning of regular expression
-    // https://eslint.org/docs/rules/no-div-regex
-    'no-div-regex': 'off',
-
-    // disallow else after a return in an if
-    // https://eslint.org/docs/rules/no-else-return
-    'no-else-return': ['error', { allowElseIf: false }],
-
-    // disallow empty functions, except for standalone funcs/arrows
-    // https://eslint.org/docs/rules/no-empty-function
-    'no-empty-function': [
-      'error',
-      {
-        allow: ['arrowFunctions', 'functions', 'methods'],
-      },
-    ],
-
-    // disallow empty destructuring patterns
-    // https://eslint.org/docs/rules/no-empty-pattern
-    'no-empty-pattern': 'error',
-
-    // disallow comparisons to null without a type-checking operator
-    // https://eslint.org/docs/rules/no-eq-null
-    'no-eq-null': 'off',
-
-    // disallow use of eval()
-    // https://eslint.org/docs/rules/no-eval
-    'no-eval': 'error',
-
-    // disallow adding to native types
-    // https://eslint.org/docs/rules/no-extend-native
-    'no-extend-native': 'error',
-
-    // disallow unnecessary function binding
-    // https://eslint.org/docs/rules/no-extra-bind
-    'no-extra-bind': 'error',
-
-    // disallow Unnecessary Labels
-    // https://eslint.org/docs/rules/no-extra-label
-    'no-extra-label': 'error',
-
-    // disallow fallthrough of case statements
-    // https://eslint.org/docs/rules/no-fallthrough
-    'no-fallthrough': 'error',
-
-    // disallow the use of leading or trailing decimal points in numeric literals
-    // https://eslint.org/docs/rules/no-floating-decimal
-    'no-floating-decimal': 'error',
-
-    // disallow reassignments of native objects or read-only globals
-    // https://eslint.org/docs/rules/no-global-assign
-    'no-global-assign': ['error', { exceptions: [] }],
-
-    // deprecated in favor of no-global-assign
-    // https://eslint.org/docs/rules/no-native-reassign
-    'no-native-reassign': 'off',
-
-    // disallow implicit type conversions
-    // https://eslint.org/docs/rules/no-implicit-coercion
-    'no-implicit-coercion': [
-      'off',
-      {
-        boolean: false,
-        number: true,
-        string: true,
-        allow: [],
-      },
-    ],
-
-    // disallow var and named functions in global scope
-    // https://eslint.org/docs/rules/no-implicit-globals
-    'no-implicit-globals': 'off',
-
-    // disallow use of eval()-like methods
-    // https://eslint.org/docs/rules/no-implied-eval
-    'no-implied-eval': 'error',
-
-    // disallow this keywords outside of classes or class-like objects
-    // https://eslint.org/docs/rules/no-invalid-this
-    'no-invalid-this': 'off',
-
-    // disallow usage of __iterator__ property
-    // https://eslint.org/docs/rules/no-iterator
-    'no-iterator': 'error',
-
-    // disallow use of labels for anything other than loops and switches
-    // https://eslint.org/docs/rules/no-labels
-    'no-labels': ['error', { allowLoop: false, allowSwitch: false }],
-
-    // disallow unnecessary nested blocks
-    // https://eslint.org/docs/rules/no-lone-blocks
-    'no-lone-blocks': 'error',
-
-    // disallow creation of functions within loops
-    // https://eslint.org/docs/rules/no-loop-func
-    'no-loop-func': 'error',
-
-    // disallow magic numbers
-    // https://eslint.org/docs/rules/no-magic-numbers
-    'no-magic-numbers': [
-      'off',
-      {
-        ignore: [],
-        ignoreArrayIndexes: true,
-        enforceConst: true,
-        detectObjects: false,
-      },
-    ],
-
-    // disallow use of multiple spaces
-    // https://eslint.org/docs/rules/no-multi-spaces
-    'no-multi-spaces': [
-      'error',
-      {
-        ignoreEOLComments: false,
-      },
-    ],
-
-    // disallow use of multiline strings
-    // https://eslint.org/docs/rules/no-multi-str
-    'no-multi-str': 'error',
-
-    // disallow use of new operator when not part of the assignment or comparison
-    // https://eslint.org/docs/rules/no-new
-    'no-new': 'error',
-
-    // disallow use of new operator for Function object
-    // https://eslint.org/docs/rules/no-new-func
-    'no-new-func': 'error',
-
-    // disallows creating new instances of String, Number, and Boolean
-    // https://eslint.org/docs/rules/no-new-wrappers
-    'no-new-wrappers': 'error',
-
-    // Disallow \8 and \9 escape sequences in string literals
-    // https://eslint.org/docs/rules/no-nonoctal-decimal-escape
-    'no-nonoctal-decimal-escape': 'error',
-
-    // disallow use of (old style) octal literals
-    // https://eslint.org/docs/rules/no-octal
-    'no-octal': 'error',
-
-    // disallow use of octal escape sequences in string literals, such as
-    // var foo = 'Copyright \251';
-    // https://eslint.org/docs/rules/no-octal-escape
-    'no-octal-escape': 'error',
-
-    // disallow reassignment of function parameters
-    // disallow parameter object manipulation except for specific exclusions
-    // rule: https://eslint.org/docs/rules/no-param-reassign.html
-    'no-param-reassign': [
-      'error',
-      {
-        props: true,
-        ignorePropertyModificationsFor: [
-          'acc', // for reduce accumulators
-          'accumulator', // for reduce accumulators
-          'e', // for e.returnvalue
-          'ctx', // for Koa routing
-          'context', // for Koa routing
-          'req', // for Express requests
-          'request', // for Express requests
-          'res', // for Express responses
-          'response', // for Express responses
-          '$scope', // for Angular 1 scopes
-          'staticContext', // for ReactRouter context
-        ],
-      },
-    ],
-
-    // disallow usage of __proto__ property
-    // https://eslint.org/docs/rules/no-proto
-    'no-proto': 'error',
-
-    // disallow declaring the same variable more than once
-    // https://eslint.org/docs/rules/no-redeclare
-    'no-redeclare': 'error',
-
-    // disallow certain object properties
-    // https://eslint.org/docs/rules/no-restricted-properties
-    'no-restricted-properties': [
-      'error',
-      {
-        object: 'arguments',
-        property: 'callee',
-        message: 'arguments.callee is deprecated',
-      },
-      {
-        object: 'global',
-        property: 'isFinite',
-        message: 'Please use Number.isFinite instead',
-      },
-      {
-        object: 'self',
-        property: 'isFinite',
-        message: 'Please use Number.isFinite instead',
-      },
-      {
-        object: 'window',
-        property: 'isFinite',
-        message: 'Please use Number.isFinite instead',
-      },
-      {
-        object: 'global',
-        property: 'isNaN',
-        message: 'Please use Number.isNaN instead',
-      },
-      {
-        object: 'self',
-        property: 'isNaN',
-        message: 'Please use Number.isNaN instead',
-      },
-      {
-        object: 'window',
-        property: 'isNaN',
-        message: 'Please use Number.isNaN instead',
-      },
-      {
-        property: '__defineGetter__',
-        message: 'Please use Object.defineProperty instead.',
-      },
-      {
-        property: '__defineSetter__',
-        message: 'Please use Object.defineProperty instead.',
-      },
-      {
-        object: 'Math',
-        property: 'pow',
-        message: 'Use the exponentiation operator (**) instead.',
-      },
-    ],
-
-    // disallow use of assignment in return statement
-    // https://eslint.org/docs/rules/no-return-assign
-    'no-return-assign': ['error', 'always'],
-
-    // disallow redundant `return await`
-    // https://eslint.org/docs/rules/no-return-await
-    'no-return-await': 'error',
-
-    // disallow use of `javascript:` urls.
-    // https://eslint.org/docs/rules/no-script-url
-    'no-script-url': 'error',
-
-    // disallow self assignment
-    // https://eslint.org/docs/rules/no-self-assign
-    'no-self-assign': [
-      'error',
-      {
-        props: true,
-      },
-    ],
-
-    // disallow comparisons where both sides are exactly the same
-    // https://eslint.org/docs/rules/no-self-compare
-    'no-self-compare': 'error',
-
-    // disallow use of comma operator
-    // https://eslint.org/docs/rules/no-sequences
-    'no-sequences': 'error',
-
-    // restrict what can be thrown as an exception
-    // https://eslint.org/docs/rules/no-throw-literal
-    'no-throw-literal': 'error',
-
-    // disallow unmodified conditions of loops
-    // https://eslint.org/docs/rules/no-unmodified-loop-condition
-    'no-unmodified-loop-condition': 'off',
-
-    // disallow usage of expressions in statement position
-    // https://eslint.org/docs/rules/no-unused-expressions
-    'no-unused-expressions': [
-      'error',
-      {
-        allowShortCircuit: false,
-        allowTernary: false,
-        allowTaggedTemplates: false,
-      },
-    ],
-
-    // disallow unused labels
-    // https://eslint.org/docs/rules/no-unused-labels
-    'no-unused-labels': 'error',
-
-    // disallow unnecessary .call() and .apply()
-    // https://eslint.org/docs/rules/no-useless-call
-    'no-useless-call': 'off',
-
-    // Disallow unnecessary catch clauses
-    // https://eslint.org/docs/rules/no-useless-catch
-    'no-useless-catch': 'error',
-
-    // disallow useless string concatenation
-    // https://eslint.org/docs/rules/no-useless-concat
-    'no-useless-concat': 'error',
-
-    // disallow unnecessary string escaping
-    // https://eslint.org/docs/rules/no-useless-escape
-    'no-useless-escape': 'error',
-
-    // disallow redundant return; keywords
-    // https://eslint.org/docs/rules/no-useless-return
-    'no-useless-return': 'error',
-
-    // disallow use of void operator
-    // https://eslint.org/docs/rules/no-void
-    'no-void': 'error',
-
-    // disallow usage of configurable warning terms in comments: e.g. todo
-    // https://eslint.org/docs/rules/no-warning-comments
-    'no-warning-comments': ['off', { terms: ['todo', 'fixme', 'xxx'], location: 'start' }],
-
-    // disallow use of the with statement
-    // https://eslint.org/docs/rules/no-with
-    'no-with': 'error',
-
-    // require using Error objects as Promise rejection reasons
-    // https://eslint.org/docs/rules/prefer-promise-reject-errors
-    'prefer-promise-reject-errors': ['error', { allowEmptyReject: true }],
-
-    // Suggest using named capture group in regular expression
-    // https://eslint.org/docs/rules/prefer-named-capture-group
-    'prefer-named-capture-group': 'off',
-
-    // https://eslint.org/docs/rules/prefer-regex-literals
-    'prefer-regex-literals': [
-      'error',
-      {
-        disallowRedundantWrapping: true,
-      },
-    ],
-
-    // require use of the second argument for parseInt()
-    // https://eslint.org/docs/rules/radix
-    radix: 'error',
-
-    // require `await` in `async function` (note: this is a horrible rule that should never be used)
-    // https://eslint.org/docs/rules/require-await
-    'require-await': 'off',
-
-    // Enforce the use of u flag on RegExp
-    // https://eslint.org/docs/rules/require-unicode-regexp
-    'require-unicode-regexp': 'off',
-
-    // requires to declare all vars on top of their containing scope
-    // https://eslint.org/docs/rules/vars-on-top
-    'vars-on-top': 'error',
-
-    // require immediate function invocation to be wrapped in parentheses
-    // https://eslint.org/docs/rules/wrap-iife.html
-    'wrap-iife': ['error', 'outside', { functionPrototypeMethods: false }],
-
-    // require or disallow Yoda conditions
-    // https://eslint.org/docs/rules/yoda
-    yoda: 'error',
-  },
-};

package/eslint9/lib/configs/base/errors.js

@@ -1,186 +0,0 @@
-module.exports = {
-  rules: {
-    // Enforce “for” loop update clause moving the counter in the right direction
-    // https://eslint.org/docs/rules/for-direction
-    'for-direction': 'error',
-
-    // Enforces that a return statement is present in property getters
-    // https://eslint.org/docs/rules/getter-return
-    'getter-return': ['error', { allowImplicit: true }],
-
-    // disallow using an async function as a Promise executor
-    // https://eslint.org/docs/rules/no-async-promise-executor
-    'no-async-promise-executor': 'error',
-
-    // Disallow await inside of loops
-    // https://eslint.org/docs/rules/no-await-in-loop
-    'no-await-in-loop': 'error',
-
-    // Disallow comparisons to negative zero
-    // https://eslint.org/docs/rules/no-compare-neg-zero
-    'no-compare-neg-zero': 'error',
-
-    // disallow assignment in conditional expressions
-    'no-cond-assign': ['error', 'always'],
-
-    // disallow use of console
-    'no-console': 'warn',
-
-    // disallow use of constant expressions in conditions
-    'no-constant-condition': 'warn',
-
-    // disallow control characters in regular expressions
-    'no-control-regex': 'error',
-
-    // disallow use of debugger
-    'no-debugger': 'error',
-
-    // disallow duplicate arguments in functions
-    'no-dupe-args': 'error',
-
-    // Disallow duplicate conditions in if-else-if chains
-    // https://eslint.org/docs/rules/no-dupe-else-if
-    'no-dupe-else-if': 'error',
-
-    // disallow duplicate keys when creating object literals
-    'no-dupe-keys': 'error',
-
-    // disallow a duplicate case label.
-    'no-duplicate-case': 'error',
-
-    // disallow empty statements
-    'no-empty': 'error',
-
-    // disallow the use of empty character classes in regular expressions
-    'no-empty-character-class': 'error',
-
-    // disallow assigning to the exception in a catch block
-    'no-ex-assign': 'error',
-
-    // disallow double-negation boolean casts in a boolean context
-    // https://eslint.org/docs/rules/no-extra-boolean-cast
-    'no-extra-boolean-cast': 'error',
-
-    // disallow unnecessary parentheses
-    // https://eslint.org/docs/rules/no-extra-parens
-    'no-extra-parens': [
-      'off',
-      'all',
-      {
-        conditionalAssign: true,
-        nestedBinaryExpressions: false,
-        returnAssign: false,
-        ignoreJSX: 'all', // delegate to eslint-plugin-react
-        enforceForArrowConditionals: false,
-      },
-    ],
-
-    // disallow unnecessary semicolons
-    'no-extra-semi': 'error',
-
-    // disallow overwriting functions written as function declarations
-    'no-func-assign': 'error',
-
-    // https://eslint.org/docs/rules/no-import-assign
-    'no-import-assign': 'error',
-
-    // disallow function or variable declarations in nested blocks
-    'no-inner-declarations': 'error',
-
-    // disallow invalid regular expression strings in the RegExp constructor
-    'no-invalid-regexp': 'error',
-
-    // disallow irregular whitespace outside of strings and comments
-    'no-irregular-whitespace': 'error',
-
-    // Disallow Number Literals That Lose Precision
-    // https://eslint.org/docs/rules/no-loss-of-precision
-    'no-loss-of-precision': 'error',
-
-    // Disallow characters which are made with multiple code points in character class syntax
-    // https://eslint.org/docs/rules/no-misleading-character-class
-    'no-misleading-character-class': 'error',
-
-    // disallow the use of object properties of the global object (Math and JSON) as functions
-    'no-obj-calls': 'error',
-
-    // Disallow returning values from Promise executor functions
-    // https://eslint.org/docs/rules/no-promise-executor-return
-    'no-promise-executor-return': 'error',
-
-    // disallow use of Object.prototypes builtins directly
-    // https://eslint.org/docs/rules/no-prototype-builtins
-    'no-prototype-builtins': 'error',
-
-    // disallow multiple spaces in a regular expression literal
-    'no-regex-spaces': 'error',
-
-    // Disallow returning values from setters
-    // https://eslint.org/docs/rules/no-setter-return
-    'no-setter-return': 'error',
-
-    // disallow sparse arrays
-    'no-sparse-arrays': 'error',
-
-    // Disallow template literal placeholder syntax in regular strings
-    // https://eslint.org/docs/rules/no-template-curly-in-string
-    'no-template-curly-in-string': 'error',
-
-    // Avoid code that looks like two expressions but is actually one
-    // https://eslint.org/docs/rules/no-unexpected-multiline
-    'no-unexpected-multiline': 'error',
-
-    // disallow unreachable statements after a return, throw, continue, or break statement
-    'no-unreachable': 'error',
-
-    // Disallow loops with a body that allows only one iteration
-    // https://eslint.org/docs/rules/no-unreachable-loop
-    'no-unreachable-loop': [
-      'error',
-      {
-        ignore: [], // WhileStatement, DoWhileStatement, ForStatement, ForInStatement, ForOfStatement
-      },
-    ],
-
-    // disallow return/throw/break/continue inside finally blocks
-    // https://eslint.org/docs/rules/no-unsafe-finally
-    'no-unsafe-finally': 'error',
-
-    // disallow negating the left operand of relational operators
-    // https://eslint.org/docs/rules/no-unsafe-negation
-    'no-unsafe-negation': 'error',
-
-    // disallow use of optional chaining in contexts where the undefined value is not allowed
-    // https://eslint.org/docs/rules/no-unsafe-optional-chaining
-    'no-unsafe-optional-chaining': ['error', { disallowArithmeticOperators: true }],
-
-    // Disallow Unused Private Class Members
-    // https://eslint.org/docs/rules/no-unused-private-class-members
-    // TODO: enable once eslint 7 is dropped (which is semver-major)
-    'no-unused-private-class-members': 'off',
-
-    // Disallow useless backreferences in regular expressions
-    // https://eslint.org/docs/rules/no-useless-backreference
-    'no-useless-backreference': 'error',
-
-    // disallow negation of the left operand of an in expression
-    // deprecated in favor of no-unsafe-negation
-    'no-negated-in-lhs': 'off',
-
-    // Disallow assignments that can lead to race conditions due to usage of await or yield
-    // https://eslint.org/docs/rules/require-atomic-updates
-    // note: not enabled because it is very buggy
-    'require-atomic-updates': 'off',
-
-    // disallow comparisons with the value NaN
-    'use-isnan': 'error',
-
-    // ensure JSDoc comments are valid
-    // https://eslint.org/docs/rules/valid-jsdoc
-    'valid-jsdoc': 'off',
-
-    // ensure that the results of typeof are compared against a valid string
-    // https://eslint.org/docs/rules/valid-typeof
-    'valid-typeof': ['error', { requireStringLiterals: true }],
-  },
-};