Skip to content

Convert prefer-readonly to @typescript-eslint/prefer-readonly #309

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
cheeZery opened this issue Dec 30, 2019 · 2 comments
Closed

Convert prefer-readonly to @typescript-eslint/prefer-readonly #309

cheeZery opened this issue Dec 30, 2019 · 2 comments
Labels
good first issue Good for newcomers; welcome aboard! status: accepting prs Please, send in a PR to resolve this! ✨ type: bug Something isn't working

Comments

@cheeZery
Copy link

🐛 Bug Report

  • tslint-to-eslint-config version: 0.5.0
  • ESLint version: 6.8.0
  • Node version: 11.10.0

Actual Behavior

TSLint rule prefer-readonly will be converted to ESLint's prefer-readonly (see #181 and #191), but alas that rule doesn't exist in ESLint. As stated here prefer-readonly should be converted to @typescript-eslint/prefer-readonly.

Expected Behavior

TSLint rule prefer-readonly will be converted to @typescript-eslint/eslint-plugin's rule @typescript-eslint/prefer-readonly.

@JoshuaKGoldberg JoshuaKGoldberg added good first issue Good for newcomers; welcome aboard! status: accepting prs Please, send in a PR to resolve this! ✨ type: bug Something isn't working labels Dec 31, 2019
@ghost
Copy link

ghost commented Jan 17, 2020

closed with #310 😉

@JoshuaKGoldberg
Copy link
Member

Aha, thanks! 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers; welcome aboard! status: accepting prs Please, send in a PR to resolve this! ✨ type: bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants