typescript-eslint/tslint-to-eslint-config

Cleanup: standardize ruleConverters/* paths internally

JoshuaKGoldberg opened this issue · 2 comments

JoshuaKGoldberg commented

Overview

Following up on https://github.com/typescript-eslint/tslint-to-eslint-config/pull/1194/files#r698063081, there's not a standard for how the sub-folders corresponding to third party plugins are named.

Actual Behavior

One folder is codelyzer (the old name), another eslint-plugin-react...

Expected Behavior

I'd expect these to generally be named after the old (TSLint) names...?

JoshuaKGoldberg commented

Actually, I just realized having plugin-specific folder names isn't going to work. Some rules are duplicated across plugins! no-constant-condition, for example, exists in both tslint-microsoft-contrib and tslint-eslint-rules.

I'll send a PR to remove the plugin names so everything is purely alphabetical by linter rule name.

  • 👍1
JoshuaKGoldberg commented

FYI @rafaelss95 - Apologies for the change of heart so soon after mentioning this issue.

  • 👍1
  • 👀1