Description
This is a follow-up issue of this PR: #250.
It is only valid if this branch gets merged.
Original discussion #250 (comment):
Reading through the branch, the difference between general linter config names (e.g. convertConfig) and their settings file equivalents (e.g. convertSettingsConfig) was a little hard to keep track of. That seems like something good for a followup issue+PR as well to clarify though, to keep the PR from getting too massive...
🚀 Feature Request
This refactoring should rather touch the general functionality. The goal is to clarify the namings, similarities and differences of rules/linter conversions and their editor settings/configurations equivalents. Currently there seems to be a great portion of copy-paste code. The similar functionality and namings make it hard to follow the different kinds of conversions. However it may involve a greater architectural change to clean this up.
Existing Behavior
- Basically
convertConfig.ts
and everything namedrules
converts TSLint rules into ESLint rules - Similar
convertEditorConfig.ts
, everything namededitorConfig
orsettings
converts TSLint editor configuration settings into ESLint editor configuration settings
Change Proposal
- Clarify on the naming similarities and differences first: Do we talk about lint rules and editor configurations or editor settings or editor configuration settings.
- The naming and folder structure should make it easy and clear to follow the different parts.