@neolution-ch/eslint-config-neolution 1.3.0 → 2.1.0

package/CHANGELOG.md

@@ -1,52 +0,0 @@
-# Changelog
-
-All notable changes to this project will be documented in this file.
-
-The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
-and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
-
-## [Unreleased]
-
-## [1.3.0] - 2022-12-13
-
-### Changed
-
--   Disabled "@typescript-eslint/non-nullable-type-assertion-style" rule
-
-## [1.2.0] - 2022-10-27
-
-### Added
-
--   Added typescript to the import/resolver
--   Added deprecated React types to the "ban-types" rule
--   Added "complexity" rule with a maximum of 20
--   Added "max-lines" rule with a maximum of 300 lines
-
-### Changed
-
--   Updated "no-empty-function" rule to not allow any empty function
--   Updated "quotes" rule by using the typescript version
--   Updated "react/no-unstable-nested-components" rule to allow component creation inside component props
-
-## [1.1.0] - 2022-10-17
-
-### Added
-
--   Added MIT license
--   Enable react-hooks rules
-
-## [1.0.0] - 2022-10-06
-
-### Added
-
--   Initial release 
-
-[Unreleased]: https://github.com/neolution-ch/eslint-config-neolution/compare/1.3.0...HEAD
-
-[1.3.0]: https://github.com/neolution-ch/eslint-config-neolution/compare/1.2.0...1.3.0
-
-[1.2.0]: https://github.com/neolution-ch/eslint-config-neolution/compare/1.1.0...1.2.0
-
-[1.1.0]: https://github.com/neolution-ch/eslint-config-neolution/compare/1.0.0...1.1.0
-
-[1.0.0]: https://github.com/neolution-ch/eslint-config-neolution/compare/5f308ef87fa2a779e56cb6af4510baf6e2deeb23...1.0.0

package/index.js

@@ -1,11 +0,0 @@
-module.exports = {
-  extends: [
-    "./rules/eslint",
-    "./rules/typescript",
-    "./rules/import",
-    "./rules/jsdoc",
-    "./rules/react",
-    "./rules/react-hooks",
-    "./rules/next",
-  ].map(require.resolve)
-};

package/rules/eslint.js

@@ -1,75 +0,0 @@
-module.exports = {
-  extends: [
-    "eslint-config-airbnb",
-  ].map(require.resolve),
-
-  // View link below for eslint rules documentation
-  // https://eslint.org/docs/rules/
-  rules: {
-    // The airbnb config enforce that class methods use "this", but requires changing how you call the method
-    // https://eslint.org/docs/rules/class-methods-use-this
-    "class-methods-use-this": "off",
-
-    // The airbnb config disables the complexity, but we want to limit it
-    // https://eslint.org/docs/rules/complexity
-    complexity: ["error", 20],
-
-    // The airbnb config enforces consistent return, but we got problems with some external libraries, TypeScript enforces this anyway
-    // https://eslint.org/docs/rules/consistent-return
-    "consistent-return": "off",
-
-    // The airbnb config forces unix style, but somebody also works on windows
-    // https://eslint.org/docs/rules/linebreak-style
-    "linebreak-style": "off",
-
-    // The airbnb config limits to 100 ignoring some lines, instad we allow 160 but don't ignore lines
-    // https://eslint.org/docs/rules/max-len
-    "max-len": ["error", 160, 2, {
-      ignoreComments: false,
-      ignoreTrailingComments: false,
-      ignoreUrls: false,
-      ignoreStrings: false,
-      ignoreTemplateLiterals: false,
-      ignoreRegExpLiterals: false,
-    }],
-
-    // The airbnb disables the max lines, but we want to limit it
-    // https://eslint.org/docs/rules/max-lines
-    "max-lines": ["error", {
-      max: 300,
-      skipBlankLines: true,
-      skipComments: true
-    }],
-
-    // The airbnb config disallow await inside of loops, but it cannot always be avoided
-    // https://eslint.org/docs/rules/no-await-in-loop
-    "no-await-in-loop": "off",
-
-    // The airbnb config disallow use of the continue statement, but it's ok to have it
-    // https://eslint.org/docs/rules/no-continue
-    "no-continue": "off",
-
-    // The airbnb config disallow use of unary operators (++ and --), but it's ok to have them
-    // https://eslint.org/docs/rules/no-plusplus
-    "no-plusplus": "off",
-
-    // The airbnb config also restrict ForOfStatement, but we want to use it
-    // 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: "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.",
-      },
-    ],
-  },
-};

package/rules/import.js

@@ -1,36 +0,0 @@
-module.exports = {
-  extends: [
-    "plugin:import/recommended"
-  ],
-
-  plugins: [
-    "import",
-  ],
-
-  settings: {
-    "import/resolver": {
-      node: {
-        extensions: [".js", ".jsx", ".ts", ".tsx"],
-      },
-      typescript: {
-        alwaysTryTypes: true,
-      },
-    },
-  },
-
-  // View link below for import rules documentation
-  // https://github.com/import-js/eslint-plugin-import/tree/main/docs/rules
-  rules: {
-    // same as airbnb but with ts/tsx extensions
-    "import/extensions": [
-      "error",
-      "ignorePackages",
-      {
-        js: "never",
-        jsx: "never",
-        ts: "never",
-        tsx: "never"
-      }
-    ]
-  },
-};

package/rules/jsdoc.js

@@ -1,25 +0,0 @@
-module.exports = {
-  extends: [
-    "plugin:jsdoc/recommended"
-  ],
-
-  plugins: [
-    "jsdoc"
-  ],
-
-  // View link below for jsdoc rules documentation
-  // https://github.com/gajus/eslint-plugin-jsdoc#user-content-eslint-plugin-jsdoc-rules
-  rules: {
-    // Do not enforce the empty line after the description block
-    // https://github.com/gajus/eslint-plugin-jsdoc#newline-after-description
-    "jsdoc/newline-after-description": "off",
-
-    // Do not require that each @param tag has a type value
-    // https://github.com/gajus/eslint-plugin-jsdoc#require-param-type
-    "jsdoc/require-param-type": "off",
-
-    // Do not require that @returns tag has a type value
-    // https://github.com/gajus/eslint-plugin-jsdoc#require-returns-type
-    "jsdoc/require-returns-type": "off"
-  }
-};

package/rules/next.js

@@ -1,17 +0,0 @@
-module.exports = {
-  extends: [
-    "plugin:@next/next/recommended"
-  ],
-
-  plugins: [
-    "@next/next",
-  ],
-
-  // View link below for next.js rules documentation
-  // https://nextjs.org/docs/basic-features/eslint#eslint-plugin
-  rules: {
-    // Disable because of this bug: https://github.com/vercel/next.js/issues/24421
-    // https://nextjs.org/docs/messages/no-img-element
-    "@next/next/no-img-element": "off",
-  },
-};

package/rules/react-hooks.js

@@ -1,10 +0,0 @@
-module.exports = {
-  extends: [
-    "eslint-config-airbnb/hooks",
-  ].map(require.resolve),
-
-  // View link below for react-hooks rules documentation
-  // https://reactjs.org/docs/hooks-rules.html
-  rules: {
-  },
-};

package/rules/react.js

@@ -1,32 +0,0 @@
-module.exports = {
-  // View link below for react rules documentation
-  // https://github.com/jsx-eslint/eslint-plugin-react#list-of-supported-rules
-  rules: {
-    // The airbnb config forces the destructuring, but it's not always desirable
-    // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/destructuring-assignment.md
-    "react/destructuring-assignment": "off",
-
-    // The airbnb config forces function-expressions, but we prefer arrow-functions
-    // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/function-component-definition.md
-    "react/function-component-definition": ["error", {
-      namedComponents: "arrow-function",
-      unnamedComponents: "arrow-function",
-    }],
-
-    // The airbnb config forces only .jsx, but we support also .tsx
-    // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/jsx-filename-extension.md
-    "react/jsx-filename-extension": ["error", { extensions: [".jsx", ".tsx"] }],
-
-    // The airbnb config forces one expression per line, but it's not always desirable
-    // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/jsx-one-expression-per-line.md
-    "react/jsx-one-expression-per-line": "off",
-
-    // The airbnb config didn't allow component creation inside component props, but we want to allow it
-    // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/no-unstable-nested-components.md
-    "react/no-unstable-nested-components": ["error", { allowAsProps: true }],
-
-    // The airbnb config forces a defaultProps definition for every prop, but we don't want it
-    // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/require-default-props.md
-    "react/require-default-props": "off",
-  },
-};

package/rules/typescript.js

@@ -1,214 +0,0 @@
-module.exports = {
-  extends: [
-    "plugin:@typescript-eslint/all"
-  ],
-
-  plugins: [
-    "@typescript-eslint"
-  ],
-
-  parser: "@typescript-eslint/parser",
-
-  parserOptions: {
-    project: "tsconfig.json"
-  },
-
-  // View link below for typescript rules documentation
-  // https://typescript-eslint.io/rules/
-  rules: {
-    // Disallow certain types
-    // https://typescript-eslint.io/rules/ban-types/
-    "@typescript-eslint/ban-types": ["error",{
-      types: {
-        "React.StatelessComponent": { message: "Deprecated: Do not use.", fixWith: "React.ReactElement" },
-        StatelessComponent: { message: "Deprecated: Do not use.", fixWith: "ReactElement" },
-        "React.FC": { message: "Please use ReactElement + PropsWithChildren", fixWith: "ReactElement" },
-        FC: { message: "Please use ReactElement + PropsWithChildren", fixWith: "ReactElement" },
-      },
-      extendDefaults: true,
-    }],
-
-    // Enforce one true brace style (same as eslint-config-airbnb-base)
-    // https://typescript-eslint.io/rules/brace-style
-    "@typescript-eslint/brace-style": ["error", "1tbs", { allowSingleLine: true }],
-
-    // Require trailing commas in multiline object literals (same as eslint-config-airbnb-base)
-    // https://typescript-eslint.io/rules/comma-dangle
-    "@typescript-eslint/comma-dangle": ["error", "always-multiline"],
-
-    // Enforce spacing before and after comma (same as eslint-config-airbnb-base)
-    // https://typescript-eslint.io/rules/comma-spacing
-    "@typescript-eslint/comma-spacing": ["error", { before: false, after: true }],
-
-    // Encourages use of dot notation whenever possible (same as eslint-config-airbnb-base)
-    // https://typescript-eslint.io/rules/dot-notation
-    "@typescript-eslint/dot-notation": ["error", { allowKeywords: true }],
-
-    // This option sets a specific tab width for your code (same as eslint-config-airbnb-base)
-    // https://typescript-eslint.io/rules/indent
-    "@typescript-eslint/indent": ["error", 2, {
-      SwitchCase: 1,
-      VariableDeclarator: 1,
-      outerIIFEBody: 1,
-      // MemberExpression: null,
-      FunctionDeclaration: {
-        parameters: 1,
-        body: 1
-      },
-      FunctionExpression: {
-        parameters: 1,
-        body: 1
-      },
-      CallExpression: {
-        arguments: 1
-      },
-      ArrayExpression: 1,
-      ObjectExpression: 1,
-      ImportDeclaration: 1,
-      flatTernaryExpressions: false,
-      // list derived from https://github.com/benjamn/ast-types/blob/HEAD/def/jsx.js
-      ignoredNodes: [
-        "JSXElement",
-        "JSXElement > *",
-        "JSXAttribute",
-        "JSXIdentifier",
-        "JSXNamespacedName",
-        "JSXMemberExpression",
-        "JSXSpreadAttribute",
-        "JSXExpressionContainer",
-        "JSXOpeningElement",
-        "JSXClosingElement",
-        "JSXFragment",
-        "JSXOpeningFragment",
-        "JSXClosingFragment",
-        "JSXText",
-        "JSXEmptyExpression",
-        "JSXSpreadChild"
-      ],
-      ignoreComments: false
-    }],
-
-    // Enforce or disallow variable initializations at definition (same as eslint-config-airbnb-base)
-    // https://typescript-eslint.io/rules/init-declarations
-    "@typescript-eslint/init-declarations": "off",
-
-    // Require a space before & after certain keywords (same as eslint-config-airbnb-base)
-    // https://typescript-eslint.io/rules/keyword-spacing
-    "@typescript-eslint/keyword-spacing": ["error", {
-      before: true,
-      after: true,
-      overrides: {
-        return: { after: true },
-        throw: { after: true },
-        case: { after: true }
-      }
-    }],
-
-    // Require or disallow an empty line between class members (same as eslint-config-airbnb-base)
-    // https://typescript-eslint.io/rules/lines-between-class-members
-    "@typescript-eslint/lines-between-class-members": ["error", "always", { exceptAfterSingleLine: false }],
-
-    // Force camelCase except for some cases
-    // https://typescript-eslint.io/rules/naming-convention
-    "@typescript-eslint/naming-convention": ["error", {
-      selector: "default",
-      format: ["camelCase"],
-      leadingUnderscore: "allow",
-    },
-    {
-      selector: ["typeLike", "accessor", "enumMember"],
-      format: ["PascalCase"],
-    }],
-
-    // Disallow empty functions
-    // https://typescript-eslint.io/rules/no-empty-function
-    "@typescript-eslint/no-empty-function": ["error", {
-      allow: [],
-    }],
-
-    // Disallow unnecessary parentheses (same as eslint-config-airbnb-base)
-    // https://typescript-eslint.io/rules/no-extra-parens
-    "@typescript-eslint/no-extra-parens": ["off", "all", {
-      conditionalAssign: true,
-      nestedBinaryExpressions: false,
-      returnAssign: false,
-      ignoreJSX: "all", // delegate to eslint-plugin-react
-      enforceForArrowConditionals: false,
-    }],
-
-    // Disallow classes used as namespaces
-    // https://typescript-eslint.io/rules/no-extraneous-class
-    "@typescript-eslint/no-extraneous-class": "off",
-
-    // Disallow void type outside of generic or return types
-    // https://typescript-eslint.io/rules/no-invalid-void-type
-    "@typescript-eslint/no-invalid-void-type": "off",
-
-    // Disallow magic numbers (same as eslint-config-airbnb-base)
-    // https://typescript-eslint.io/rules/no-magic-numbers
-    "@typescript-eslint/no-magic-numbers": ["off", {
-      ignore: [],
-      ignoreArrayIndexes: true,
-      enforceConst: true,
-      detectObjects: false,
-    }],
-
-    // Disallow custom TypeScript modules and namespaces
-    // https://typescript-eslint.io/rules/no-namespace
-    "@typescript-eslint/no-namespace": ["error", {
-      allowDeclarations: true,
-      allowDefinitionFiles: true,
-    }],
-
-    // Disallow type aliases
-    // https://typescript-eslint.io/rules/no-type-alias
-    "@typescript-eslint/no-type-alias": "off",
-
-    // Disallow declaration of variables that are not used in the code (same as eslint-config-airbnb-base)
-    // https://typescript-eslint.io/rules/no-unused-vars
-    "@typescript-eslint/no-unused-vars": ["error", { vars: "all", args: "after-used", ignoreRestSiblings: true }],
-
-    // Disable forcing non-null assertions over explicit type casts
-    // https://typescript-eslint.io/rules/non-nullable-type-assertion-style
-    "@typescript-eslint/non-nullable-type-assertion-style": "off",
-
-    // Require padding inside curly braces (same as eslint-config-airbnb-base)
-    // https://typescript-eslint.io/rules/object-curly-spacing
-    "@typescript-eslint/object-curly-spacing": ["error", "always"],
-
-    // Require or disallow padding lines between statements (same as eslint-config-airbnb-base)
-    // https://typescript-eslint.io/rules/padding-line-between-statements
-    "@typescript-eslint/padding-line-between-statements": "off",
-
-    // Require function parameters to be typed as readonly to prevent accidental mutation of inputs
-    // https://typescript-eslint.io/rules/prefer-readonly-parameter-types
-    "@typescript-eslint/prefer-readonly-parameter-types": "off",
-
-    // Enforce the consistent use of double quotes, but allow single quotes so long as the
-    // string contains a quote that would have to be escaped otherwise
-    // https://typescript-eslint.io/rules/quotes
-    "@typescript-eslint/quotes": ["error", "double", { avoidEscape: true }],
-
-    // Require `await` in `async function` (note: this is a horrible rule that should never be used) (same as eslint-config-airbnb-base)
-    // https://typescript-eslint.io/rules/require-await
-    "@typescript-eslint/require-await": "off",
-
-    // Enforce template literal expressions to be of string type, but allow numbers and booleans
-    // https://typescript-eslint.io/rules/restrict-template-expressions
-    "@typescript-eslint/restrict-template-expressions": ["error", {
-      allowNumber: true,
-      allowBoolean: true,
-      allowAny: false,
-      allowNullish: false,
-      allowRegExp: false,
-    }],
-
-    // Require or disallow space before function opening parenthesis (same as eslint-config-airbnb-base)
-    // https://typescript-eslint.io/rules/space-before-function-paren
-    "@typescript-eslint/space-before-function-paren": ["error", {
-      anonymous: "always",
-      named: "never",
-      asyncArrow: "always"
-    }],
-  },
-};