-
Notifications
You must be signed in to change notification settings - Fork 308
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
v6: Migrate from tslint to eslint (next branch) #3065
Comments
Hey, I was looking into this issue and created a new branch already (with an MR pointing to my own fork) I used an AI to convert the existing tslint config to eslint, so hopefully it kinda aligns with the setup you had before. There's 2 things I want to check
New eslint issues
|
Hey, thanks for this!! I took a look at your branch and ran it locally, it seems good to me! Ideally, we want to match to existing format. You should create a PR and we can look to resolve rules/issues like indenting switch statements, ignoring specific files, line sizes, etc. In regards to prettier, we don't want to have an opinionated code formatter as of now, but if needed down the line, we can add it. |
Tslint is deprecated and we should migrate to eslint. V5 did not include eslint as it was too slow, but now we are seeing errors when running tslint... possibly not working.
The text was updated successfully, but these errors were encountered: