-
Notifications
You must be signed in to change notification settings - Fork 100
Closed
Labels
area: incorrect converterRule converter with incomplete, incorrect, or invalid rule names and/or argumentsRule converter with incomplete, incorrect, or invalid rule names and/or argumentsgood first issueGood for newcomers; welcome aboard!Good for newcomers; welcome aboard!status: accepting prsPlease, send in a PR to resolve this! ✨Please, send in a PR to resolve this! ✨
Description
🐛 Bug Report
tslint-to-eslint-config
version: 0.2.2- ESLint version: N/A
- Node version: N/A
Actual Behavior
Right now, the converter from TSLint's member-access
rule to ESLint's @typescript-eslint/explicit-member-visibility
does not read the TSLint rule options. It assumes there are no configuration rule options for the rule.
Expected Behavior
There are actually several options on both the TSLint and ESLint side:
- https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/explicit-member-accessibility.md
- https://palantir.github.io/tslint/rules/member-access/
The rule converter should read in those options from the TSLint rule and create the equivalent configuration (if possible) in the output ESLint settings.
Metadata
Metadata
Assignees
Labels
area: incorrect converterRule converter with incomplete, incorrect, or invalid rule names and/or argumentsRule converter with incomplete, incorrect, or invalid rule names and/or argumentsgood first issueGood for newcomers; welcome aboard!Good for newcomers; welcome aboard!status: accepting prsPlease, send in a PR to resolve this! ✨Please, send in a PR to resolve this! ✨