@cabify/eslint-config 3.0.1-beta-17 → 3.0.1-beta-19

package/configs/commonjs/strict.cjs

@@ -1,7 +0,0 @@
-module.exports = {
-  name: 'strict-cabify-eslint-config',
-  rules: {
-    // babel inserts `'use strict';` for us
-    strict: ['error', 'never'],
-  },
-};

package/configs/commonjs/style.cjs

@@ -1,319 +0,0 @@
-module.exports = {
-  name: 'style-cabify-eslint-config',
-  rules: {
-    // require camel case names
-    // TODO: semver-major (eslint 5): add ignoreDestructuring: false option
-    camelcase: [
-      'error',
-      {
-        properties: 'never',
-        allow: [
-          'UNSAFE_componentDidMount',
-          'UNSAFE_componentWillReceiveProps',
-          'UNSAFE_componentWillUpdate',
-          'UNSAFE_componentWillMount',
-        ],
-      },
-    ],
-    // enforce or disallow capitalization of the first letter of a comment
-    // https://eslint.org/docs/rules/capitalized-comments
-    'capitalized-comments': [
-      'off',
-      'never',
-      {
-        line: {
-          ignorePattern: '.*',
-          ignoreInlineComments: true,
-          ignoreConsecutiveComments: true,
-        },
-        block: {
-          ignorePattern: '.*',
-          ignoreInlineComments: true,
-          ignoreConsecutiveComments: true,
-        },
-      },
-    ],
-
-    // enforces consistent naming when capturing the current execution context
-    'consistent-this': 'off',
-
-    // requires function names to match the name of the variable or property to which they are
-    // assigned
-    // https://eslint.org/docs/rules/func-name-matching
-    // TODO: semver-major (eslint 5): add considerPropertyDescriptor: true
-    'func-name-matching': [
-      'off',
-      'always',
-      {
-        includeCommonJSModuleExports: false,
-      },
-    ],
-
-    // require function expressions to have a name
-    // https://eslint.org/docs/rules/func-names
-    'func-names': 'warn',
-
-    // enforces use of function declarations or expressions
-    // https://eslint.org/docs/rules/func-style
-    // TODO: enable
-    'func-style': ['off', 'expression'],
-
-    // Blacklist certain identifiers to prevent them being used
-    // https://eslint.org/docs/rules/id-blacklist
-    'id-blacklist': 'off',
-
-    // this option enforces minimum and maximum identifier lengths
-    // (variable names, property names etc.)
-    'id-length': 'off',
-
-    // require identifiers to match the provided regular expression
-    'id-match': 'off',
-
-    // enforce position of line comments
-    // https://eslint.org/docs/rules/line-comment-position
-    // TODO: enable?
-    'line-comment-position': [
-      'off',
-      {
-        position: 'above',
-        ignorePattern: '',
-        applyDefaultPatterns: true,
-      },
-    ],
-
-    // require or disallow an empty line between class members
-    // https://eslint.org/docs/rules/lines-between-class-members
-    'lines-between-class-members': [
-      'error',
-      'always',
-      { exceptAfterSingleLine: false },
-    ],
-
-    // enforces empty lines around comments
-    'lines-around-comment': 'off',
-
-    // require or disallow newlines around directives
-    // https://eslint.org/docs/rules/lines-around-directive
-    'lines-around-directive': [
-      'error',
-      {
-        before: 'always',
-        after: 'always',
-      },
-    ],
-
-    // specify the maximum depth that blocks can be nested
-    'max-depth': ['off', 4],
-
-    // specify the max number of lines in a file
-    // https://eslint.org/docs/rules/max-lines
-    'max-lines': [
-      'off',
-      {
-        max: 300,
-        skipBlankLines: true,
-        skipComments: true,
-      },
-    ],
-
-    // enforce a maximum function length
-    // https://eslint.org/docs/rules/max-lines-per-function
-    'max-lines-per-function': [
-      'off',
-      {
-        max: 50,
-        skipBlankLines: true,
-        skipComments: true,
-        IIFEs: true,
-      },
-    ],
-
-    // specify the maximum depth callbacks can be nested
-    'max-nested-callbacks': 'off',
-
-    // limits the number of parameters that can be used in the function declaration.
-    'max-params': ['off', 3],
-
-    // specify the maximum number of statement allowed in a function
-    'max-statements': ['off', 10],
-
-    // restrict the number of statements per line
-    // https://eslint.org/docs/rules/max-statements-per-line
-    'max-statements-per-line': ['off', { max: 1 }],
-
-    // enforce a particular style for multiline comments
-    // https://eslint.org/docs/rules/multiline-comment-style
-    'multiline-comment-style': ['off', 'starred-block'],
-
-    // require a capital letter for constructors
-    'new-cap': [
-      'error',
-      {
-        newIsCap: true,
-        newIsCapExceptions: [],
-        capIsNew: false,
-        capIsNewExceptions: [
-          'Immutable.Map',
-          'Immutable.Set',
-          'Immutable.List',
-        ],
-      },
-    ],
-
-    // disallow use of the Array constructor
-    'no-array-constructor': 'error',
-
-    // disallow use of bitwise operators
-    // https://eslint.org/docs/rules/no-bitwise
-    'no-bitwise': 'error',
-
-    // disallow use of the continue statement
-    // https://eslint.org/docs/rules/no-continue
-    'no-continue': 'error',
-
-    // disallow comments inline after code
-    'no-inline-comments': 'off',
-
-    // disallow if as the only statement in an else block
-    // https://eslint.org/docs/rules/no-lonely-if
-    'no-lonely-if': 'error',
-
-    // disallow un-paren'd mixes of different operators
-    // https://eslint.org/docs/rules/no-mixed-operators
-    'no-mixed-operators': [
-      'error',
-      {
-        // the list of arthmetic groups disallows mixing `%` and `**`
-        // with other arithmetic operators.
-        groups: [
-          ['%', '**'],
-          ['%', '+'],
-          ['%', '-'],
-          ['%', '*'],
-          ['%', '/'],
-          ['**', '+'],
-          ['**', '-'],
-          ['**', '*'],
-          ['**', '/'],
-          ['&', '|', '^', '~', '<<', '>>', '>>>'],
-          ['==', '!=', '===', '!==', '>', '>=', '<', '<='],
-          ['&&', '||'],
-          ['in', 'instanceof'],
-        ],
-        allowSamePrecedence: true,
-      },
-    ],
-
-    // disallow use of chained assignment expressions
-    // https://eslint.org/docs/rules/no-multi-assign
-    'no-multi-assign': ['error'],
-
-    // disallow negated conditions
-    // https://eslint.org/docs/rules/no-negated-condition
-    'no-negated-condition': 'off',
-
-    // disallow nested ternary expressions
-    'no-nested-ternary': 'error',
-
-    // disallow use of the Object constructor
-    'no-new-object': 'error',
-
-    // disallow use of unary operators, ++ and --
-    // https://eslint.org/docs/rules/no-plusplus
-    'no-plusplus': 'off',
-
-    // disallow certain syntax forms
-    // https://eslint.org/docs/rules/no-restricted-syntax
-    'no-restricted-syntax': [
-      'error',
-      {
-        selector: 'ForInStatement',
-        message:
-          'for..in loops iterate over the entire prototype chain, which is virtually never what you want. Use Object.{keys,values,entries}, and iterate over the resulting array.',
-      },
-      {
-        selector: 'ForOfStatement',
-        message:
-          'iterators/generators require regenerator-runtime, which is too heavyweight for this guide to allow them. Separately, loops should be avoided in favor of array iterations.',
-      },
-      {
-        selector: 'LabeledStatement',
-        message:
-          'Labels are a form of GOTO; using them makes code confusing and hard to maintain and understand.',
-      },
-      {
-        selector: 'WithStatement',
-        message:
-          '`with` is disallowed in strict mode because it makes code impossible to predict and optimize.',
-      },
-    ],
-
-    // disallow the use of ternary operators
-    'no-ternary': 'off',
-
-    // disallow dangling underscores in identifiers
-    // https://eslint.org/docs/rules/no-underscore-dangle
-    'no-underscore-dangle': [
-      'error',
-      {
-        allow: ['__REDUX_DEVTOOLS_EXTENSION_COMPOSE__', '__html'],
-        allowAfterThis: false,
-        allowAfterSuper: false,
-        enforceInMethodNames: true,
-      },
-    ],
-
-    // disallow the use of Boolean literals in conditional expressions
-    // also, prefer `a || b` over `a ? a : b`
-    // https://eslint.org/docs/rules/no-unneeded-ternary
-    'no-unneeded-ternary': ['error', { defaultAssignment: false }],
-
-    // enforce the location of single-line statements
-    // https://eslint.org/docs/rules/nonblock-statement-body-position
-    'nonblock-statement-body-position': ['error', 'beside', { overrides: {} }],
-
-    // allow just one var statement per function
-    'one-var': ['error', 'never'],
-
-    // require assignment operator shorthand where possible or prohibit it entirely
-    // https://eslint.org/docs/rules/operator-assignment
-    'operator-assignment': ['error', 'always'],
-
-    // Requires operator at the beginning of the line in multiline statements
-    // https://eslint.org/docs/rules/operator-linebreak
-    'operator-linebreak': ['error', 'before', { overrides: { '=': 'none' } }],
-
-    // Prefer use of an object spread over Object.assign
-    // https://eslint.org/docs/rules/prefer-object-spread
-    // TODO: semver-major (eslint 5): enable
-    'prefer-object-spread': 'error',
-
-    // do not require jsdoc
-    // https://eslint.org/docs/rules/require-jsdoc
-    'require-jsdoc': 'off',
-
-    // requires object keys to be sorted
-    'sort-keys': ['off', 'asc', { caseSensitive: false, natural: true }],
-
-    // sort variables within the same declaration block
-    'sort-vars': 'off',
-
-    // require or disallow a space immediately following the // or /* in a comment
-    // https://eslint.org/docs/rules/spaced-comment
-    'spaced-comment': [
-      'error',
-      'always',
-      {
-        line: {
-          exceptions: ['-', '+'],
-          markers: ['=', '!'], // space here to support sprockets directives
-        },
-        block: {
-          exceptions: ['-', '+'],
-          markers: ['=', '!'], // space here to support sprockets directives
-          balanced: true,
-        },
-      },
-    ],
-  },
-};

package/configs/commonjs/ts.cjs

@@ -1,162 +0,0 @@
-/* eslint-disable no-param-reassign */
-const path = require('path');
-// eslint-disable-next-line import/no-unresolved
-const tseslint = require('typescript-eslint');
-
-const tsCustomConfig = {
-  name: 'ts-cabify-eslint-config',
-  files: ['**/*.ts', '**/*.tsx'],
-  rules: {
-    '@typescript-eslint/restrict-template-expressions': [
-      'error',
-      { allowAny: true },
-    ],
-    '@typescript-eslint/array-type': ['error', { default: 'array' }],
-    '@typescript-eslint/consistent-type-definitions': ['error', 'interface'],
-    '@typescript-eslint/member-ordering': 'error',
-    '@typescript-eslint/explicit-function-return-type': 'off',
-    '@typescript-eslint/naming-convention': [
-      'error',
-      {
-        selector: 'default',
-        format: ['camelCase'],
-      },
-      {
-        selector: 'import',
-        format: ['camelCase', 'PascalCase', 'UPPER_CASE'],
-      },
-      {
-        // This is the 'property' group values. For some reason `property` selector is not recogniced.
-        selector: ['classProperty', 'objectLiteralProperty', 'typeProperty'],
-        format: null,
-      },
-      {
-        selector: 'variableLike',
-        format: ['camelCase', 'PascalCase'],
-      },
-      {
-        selector: 'variable',
-        format: ['camelCase', 'PascalCase', 'UPPER_CASE'],
-      },
-      {
-        // 'memberLike' group selector values without 'property' and 'enumMember'.
-        // Don't know why 'memberLike' selector it's not being overriden by 'property' and 'enumMember'
-        selector: ['parameterProperty', 'method', 'accessor'],
-        format: ['camelCase', 'PascalCase'],
-        leadingUnderscore: 'allow',
-      },
-      {
-        selector: 'memberLike',
-        modifiers: ['private'],
-        format: ['camelCase'],
-        leadingUnderscore: 'forbid',
-      },
-      {
-        selector: 'typeLike',
-        format: ['PascalCase'],
-      },
-      // Allow leading underscore for untyped parameters on Ts
-      // Also allow paramters to be PascalCase to receive React.Components as props.
-      {
-        selector: 'parameter',
-        format: ['camelCase', 'PascalCase'],
-        leadingUnderscore: 'allow',
-      },
-      // Enforce the names of generic types are at least 2 characters long
-      {
-        selector: 'typeParameter',
-        format: ['PascalCase'],
-        custom: {
-          regex: '^[A-Z][a-zA-Z]+$',
-          match: true,
-        },
-      },
-      // Enforce that interface names do not begin with an I
-      {
-        selector: 'interface',
-        format: ['PascalCase'],
-        custom: {
-          regex: '^I[A-Z]',
-          match: false,
-        },
-      },
-      // Emulate the old @typescript-eslint/class-name-casing
-      {
-        selector: 'class',
-        format: ['PascalCase'],
-        leadingUnderscore: 'forbid',
-      },
-      {
-        selector: 'enumMember',
-        format: ['UPPER_CASE', 'PascalCase'],
-      },
-    ],
-    '@typescript-eslint/no-non-null-assertion': 'off',
-    '@typescript-eslint/no-explicit-any': 'error',
-    '@typescript-eslint/no-unused-expressions': [
-      'error',
-      {
-        allowShortCircuit: true,
-        allowTernary: true,
-        allowTaggedTemplates: false,
-      },
-    ],
-    '@typescript-eslint/no-unused-vars': [
-      'error',
-      { ignoreRestSiblings: true, argsIgnorePattern: '^_' },
-    ],
-    '@typescript-eslint/no-use-before-define': [
-      'error',
-      { functions: true, classes: true, variables: true },
-    ],
-    '@typescript-eslint/no-redeclare': ['error'],
-    '@typescript-eslint/no-shadow': ['error'],
-    '@typescript-eslint/explicit-module-boundary-types': 'off', // Disabled after upgraded parser
-    '@typescript-eslint/no-unsafe-member-access': 'off', // Disabled after upgraded parser
-    '@typescript-eslint/no-unsafe-assignment': 'off', // Disabled after upgraded parser
-    '@typescript-eslint/no-unsafe-call': 'off', // Disabled after upgraded parser
-    '@typescript-eslint/no-unsafe-return': 'off', // Disabled after upgraded parser
-    camelcase: 'off', // superseeded by @typescript-eslint/naming-conventions
-    'import/no-cycle': 'off',
-    'import/no-default-export': 'error',
-    // https://github.com/typescript-eslint/typescript-eslint/blob/master/docs/getting-started/linting/FAQ.md#eslint-plugin-import
-    'import/named': 'off',
-    'import/namespace': 'off',
-    'import/default': 'off',
-    'import/no-named-as-default-member': 'off',
-    'react/sort-comp': 'off',
-    'react/prop-types': 'off', // guaranteed by TS prop interfaces
-    'react/require-default-props': 'off',
-    'react/default-props-match-prop-types': 'off',
-    'no-unused-expressions': 'off', // superseeded by @typescript-eslint/no-unused-expressions
-    'no-unused-vars': 'off', // superseeded by @typescript-eslint/no-unused-vars
-    'import/prefer-default-export': 'off',
-    'no-useless-constructor': 'off',
-    '@typescript-eslint/no-useless-constructor': 'error',
-    'no-use-before-define': 'off', // superseeded by @typescript-eslint/no-use-before-define
-    'no-redeclare': 'off', // superseeded by @typescript-eslint/no-redeclare
-    'no-shadow': 'off', // superseeded by @typescript-eslint/no-shadow
-  },
-  languageOptions: {
-    parserOptions: {
-      projectService: true,
-      tsconfigRootDir: path.resolve(process.cwd()),
-    },
-  },
-  ignores: ['*.d.ts'],
-};
-
-const tsLintConfig = tseslint.config(
-  tseslint.configs.recommended,
-  tseslint.configs.recommendedTypeChecked,
-  tsCustomConfig,
-);
-
-if (tsLintConfig.length) {
-  tsLintConfig.forEach((tsconfig) => {
-    tsconfig.files = ['**/*.ts', '**/*.tsx'];
-    tsconfig.ignores = ['**/*.d.ts'];
-  });
-}
-
-module.exports = { tsLintConfig };

package/configs/commonjs/utils.cjs

@@ -1,11 +0,0 @@
-module.exports = {
-  isPackageAvailable(packageName) {
-    try {
-      // eslint-disable-next-line global-require, import/no-dynamic-require
-      return !!require(packageName);
-      // eslint-disable-next-line no-unused-vars
-    } catch (e) {
-      return false;
-    }
-  },
-};

package/configs/commonjs/variables.cjs

@@ -1,53 +0,0 @@
-const confusingBrowserGlobals = require('confusing-browser-globals');
-
-module.exports = {
-  name: 'variables-cabify-eslint-config',
-  rules: {
-    // enforce or disallow variable initializations at definition
-    'init-declarations': 'off',
-
-    // disallow the catch clause parameter name being the same as a variable in the outer scope
-    'no-catch-shadow': 'off',
-
-    // disallow deletion of variables
-    'no-delete-var': 'error',
-
-    // disallow labels that share a name with a variable
-    // https://eslint.org/docs/rules/no-label-var
-    'no-label-var': 'error',
-
-    // disallow specific globals
-    'no-restricted-globals': ['error', 'isFinite', 'isNaN'].concat(
-      confusingBrowserGlobals,
-    ),
-
-    // disallow declaration of variables already declared in the outer scope
-    'no-shadow': 'error',
-
-    // disallow shadowing of names such as arguments
-    'no-shadow-restricted-names': 'error',
-
-    // disallow use of undeclared variables unless mentioned in a /*global */ block
-    'no-undef': 'error',
-
-    // disallow use of undefined when initializing variables
-    'no-undef-init': 'error',
-
-    // disallow use of undefined variable
-    // https://eslint.org/docs/rules/no-undefined
-    // TODO: enable?
-    'no-undefined': 'off',
-
-    // disallow declaration of variables that are not used in the code
-    'no-unused-vars': [
-      'error',
-      { vars: 'all', args: 'after-used', ignoreRestSiblings: true },
-    ],
-
-    // disallow use of variables before they are defined
-    'no-use-before-define': [
-      'error',
-      { functions: true, classes: true, variables: true },
-    ],
-  },
-};

package/recommended.cjs

@@ -1,5 +0,0 @@
-const eslintConfigPrettier = require('eslint-config-prettier');
-
-const base = require('./configs/commonjs/base.cjs');
-
-module.exports = [...base, eslintConfigPrettier];