8000 feat(eslint-plugin): remove deprecated rules (#739) · invalidred/typescript-eslint@e32c7ad · GitHub
[go: up one dir, main page]
Skip to content
8000

Commit e32c7ad

Browse files
bradzacherJamesHenry
bradzacher
authored and
JamesHenry
committed
feat(eslint-plugin): remove deprecated rules (typescript-eslint#739)
BREAKING CHANGE: removing rules
1 parent b2ca20d commit e32c7ad

File tree

9 files changed

+4
-313
lines changed
  • packages/eslint-plugin
    • < 10000 div style="width:100%;display:flex">
      ROADMAP.md
  • docs/rules
  • src
  • tests/rules

    • 9 files changed

    +4
    -313
    lines changed

    ‎packages/eslint-plugin/ROADMAP.md

    Lines changed: 4 additions & 4 deletions
    Original file line numberDiff line numberDiff line change
    @@ -26,7 +26,7 @@
    2626
    | [`no-namespace`] | ✅ | [`@typescript-eslint/no-namespace`] |
    2727
    | [`no-non-null-assertion`] | ✅ | [`@typescript-eslint/no-non-null-assertion`] |
    2828
    | [`no-parameter-reassignment`] | ✅ | [`no-param-reassign`][no-param-reassign] |
    29-
    | [`no-reference`] | ✅ | [`@typescript-eslint/no-triple-slash-reference`] |
    29+
    | [`no-reference`] | ✅ | [`@typescript-eslint/triple-slash-reference`] |
    3030
    | [`no-unnecessary-type-assertion`] | ✅ | [`@typescript-eslint/no-unnecessary-type-assertion`] |
    3131
    | [`no-var-requires`] | ✅ | [`@typescript-eslint/no-var-requires`] |
    3232
    | [`only-arrow-functions`] | 🔌 | [`prefer-arrow/prefer-arrow-functions`] |
    @@ -148,7 +148,7 @@
    148148
    | [`import-spacing`] | 🔌 | Use [Prettier] |
    149149
    | [`increment-decrement`] | 🌟 | [`no-plusplus`][no-plusplus] |
    150150
    | [`interface-name`] | ✅ | [`@typescript-eslint/interface-name-prefix`] |
    151-
    | [`interface-over-type-literal`] | ✅ | [`@typescript-eslint/prefer-interface`] |
    151+
    | [`interface-over-type-literal`] | ✅ | [`@typescript-eslint/consistent-type-definitions`] |
    152152
    | [`jsdoc-format`] | 🌓 | [`valid-jsdoc`][valid-jsdoc] or [`eslint-plugin-jsdoc`][plugin:jsdoc] |
    153153
    | [`match-default-export-name`] | 🛑 | N/A |
    154154
    | [`newline-before-return`] | 🌟 | [`padding-line-between-statements`][padding-line-between-statements] <sup>[1]</sup> |
    @@ -577,6 +577,7 @@ Relevant plugins: [`chai-expect-keywords`](https://github.com/gavinaiken/eslint-
    577577
    [`@typescript-eslint/await-thenable`]: https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/await-thenable.md
    578578
    [`@typescript-eslint/ban-types`]: https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/ban-types.md
    579579
    [`@typescript-eslint/ban-ts-ignore`]: https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/ban-ts-ignore.md
    580+
    [`@typescript-eslint/consistent-type-definitions`]: https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/consistent-type-definitions.md
    580581
    [`@typescript-eslint/explicit-member-accessibility`]: https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/explicit-member-accessibility.md
    581582
    [`@typescript-eslint/member-ordering`]: https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/member-ordering.md
    582583
    [`@typescript-eslint/no-explicit-any`]: https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/no-explicit-any.md
    @@ -586,7 +587,7 @@ Relevant plugins: [`chai-expect-keywords`](https://github.com/gavinaiken/eslint-
    586587
    [`@typescript-eslint/promise-function-async`]: https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/promise-function-async.md
    587588
    [`@typescript-eslint/no-namespace`]: https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/no-namespace.md
    588589
    [`@typescript-eslint/no-non-null-assertion`]: https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/no-non-null-assertion.md
    589-
    [`@typescript-eslint/no-triple-slash-reference`]: https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/no-triple-slash-reference.md
    590+
    [`@typescript-eslint/triple-slash-reference`]: https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/triple-slash-reference.md
    590591
    [`@typescript-eslint/unbound-method`]: https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/unbound-method.md
    591592
    [`@typescript-eslint/no-unnecessary-type-assertion`]: https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/no-unnecessary-type-assertion.md
    592593
    [`@typescript-eslint/no-var-requires`]: https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/no-var-requires.md
    @@ -608,7 +609,6 @@ Relevant plugins: [`chai-expect-keywords`](https://github.com/gavinaiken/eslint-
    608609
    [`@typescript-eslint/no-parameter-properties`]: https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/no-parameter-properties.md
    609610
    [`@typescript-eslint/member-delimiter-style`]: https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/member-delimiter-style.md
    610611
    [`@typescript-eslint/prefer-for-of`]: https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/prefer-for-of.md
    611-
    [`@typescript-eslint/prefer-interface`]: https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/prefer-interface.md
    612612
    [`@typescript-eslint/no-array-constructor`]: https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/no-array-constructor.md
    613613
    [`@typescript-eslint/prefer-function-type`]: https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/prefer-function-type.md
    614614
    [`@typescript-eslint/prefer-readonly`]: https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/prefer-readonly.md

    ‎packages/eslint-plugin/docs/rules/no-triple-slash-reference.md

    Lines changed: 0 additions & 43 deletions
    This file was deleted.

    ‎packages/eslint-plugin/docs/rules/prefer-interface.md

    Lines changed: 0 additions & 36 deletions
    This file was deleted.

    ‎packages/eslint-plugin/src/configs/recommended.json

    Lines changed: 0 additions & 2 deletions
    Original file line numberDiff line numberDiff line change
    @@ -24,13 +24,11 @@
    2424
    "@typescript-eslint/no-non-null-assertion": "error",
    2525
    "@typescript-eslint/no-object-literal-type-assertion": "error",
    2626
    "@typescript-eslint/no-parameter-properties": "error",
    27-
    "@typescript-eslint/no-triple-slash-reference": "error",
    2827
    "no-unused-vars": "off",
    2928
    "@typescript-eslint/no-unused-vars": "warn",
    3029
    "no-use-before-define": "off",
    3130
    "@typescript-eslint/no-use-before-define": "error",
    3231
    "@typescript-eslint/no-var-requires": "error",
    33-
    "@typescript-eslint/prefer-interface": "error",
    3432
    "@typescript-eslint/prefer-namespace-keyword": "error",
    3533
    "@typescript-eslint/type-annotation-spacing": "error"
    3634
    }

    ‎packages/eslint-plugin/src/rules/index.ts

    Lines changed: 0 additions & 4 deletions
    Original file line numberDiff line numberDiff line change
    @@ -34,7 +34,6 @@ import noObjectLiteralTypeAssertion from './no-object-literal-type-assertion';
    3434
    import noParameterProperties from './no-parameter-properties';
    3535
    import noRequireImports from './no-require-imports';
    3636
    import noThisAlias from './no-this-alias';
    37-
    import noTripleSlashReference from './no-triple-slash-reference';
    3837
    import noTypeAlias from './no-type-alias';
    3938
    import noUnnecessaryQualifier from './no-unnecessary-qualifier';
    4039
    import noUnnecessaryTypeAssertion from './no-unnecessary-type-assertion';
    @@ -45,7 +44,6 @@ import noVarRequires from './no-var-requires';
    4544
    import preferForOf from './prefer-for-of';
    4645
    import preferFunctionType from './prefer-function-type';
    4746
    import preferIncludes from './prefer-includes';
    48-
    import preferInterface from './prefer-interface';
    4947
    import preferNamespaceKeyword from './prefer-namespace-keyword';
    5048
    import preferReadonly from './prefer-readonly';
    5149
    import preferRegexpExec from './prefer-regexp-exec';
    @@ -98,7 +96,6 @@ export default {
    9896
    'no-parameter-properties': noParameterProperties,
    9997
    'no-require-imports': noRequireImports,
    10098
    'no-this-alias': noThisAlias,
    101-
    'no-triple-slash-reference': noTripleSlashReference,
    10299
    'no-type-alias': noTypeAlias,
    103100
    'no-unnecessary-qualifier': noUnnecessaryQualifier,
    104101
    'no-unnecessary-type-assertion': noUnnecessaryTypeAssertion,
    @@ -109,7 +106,6 @@ export default {
    109106
    'prefer-for-of': preferForOf,
    110107
    'prefer-function-type': preferFunctionType,
    111108
    'prefer-includes': preferIncludes,
    112-
    'prefer-interface': preferInterface,
    113109
    'prefer-namespace-keyword': preferNamespaceKeyword,
    114110
    'prefer-readonly': preferReadonly,
    115111
    'prefer-regexp-exec': preferRegexpExec,

    ‎packages/eslint-plugin/src/rules/no-triple-slash-reference.ts

    Lines changed: 0 additions & 42 deletions
    This file was deleted.

    ‎packages/eslint-plugin/src/rules/prefer-interface.ts

    Lines changed: 0 additions & 64 deletions
    This file was deleted.

    ‎packages/eslint-plugin/tests/rules/no-triple-slash-reference.test.ts

    Lines changed: 0 additions & 45 deletions
    This file was deleted.

    ‎packages/eslint-plugin/tests/rules/prefer-interface.test.ts

    Lines changed: 0 additions & 73 deletions
    This file was deleted.

    0 commit comments

    Comments
     (0)
    0