Rules: Deprecate extension rules now duplicated by TS-syntax-aware core rules
DRANK

Before You File a Proposal Please Confirm You Have Done The Following... I have searched for related issues and found none that match my proposal. I have read the FAQ and my problem is not listed. Description Coming over from https://github.com/eslint/eslint/issues/19173: the following non-type-aware extension rules are being replaced with core ESLint rules: @typescript-eslint/class-methods-use-this @typescript-eslint/default-param-last @typescript-eslint/init-declarations @typescript-eslint/max-params @typescript-eslint/no-array-constructor @typescript-eslint/no-dupe-class-members @typescript-eslint/no-empty-function @typescript-eslint/no-invalid-this: @typescript-eslint/no-loop-func @typescript-eslint/no-loss-of-precision @typescript-eslint/no-magic-numbers @typescript-eslint/no-redeclare @typescript-eslint/no-restricted-imports @typescript-eslint/no-shadow @typescript-eslint/no-unused-expressions @typescript-eslint/no-unused-vars @typescript-eslint/no-use-before-define @typescript-eslint/no-useless-constructor We're presumably going to want to: Deprecate these extension rules for the current major version File a followup breaking change issue to remove them in the next major that includes breaking changes Proposal: let's tackle (1) once all the rules are implemented in ESLint core? That will cause less churn on our end and let us see what unexpected learnings, if any, are found in the still-pending ESLint rule updates. Additional Info 💖

github.com
Related Topics: TypeScript
1 comments