-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
Migrate to ESLint #1824
Comments
Do you have a link to that announcement ? |
Thanks. Yes I'm all for this. |
cc/ @JamesHenry Any thoughts on migration approach appreciated. |
Check out our docs on equivalents for TSLint rules for some guidance. |
@j-f1 any good standard configs you can recommend. I am not totally a fan of our current rules. |
We’ve got a |
Perhaps https://discuss.bevry.me/t/how-bevry-automates-maintenance-of-its-prolific-open-source-portfolio/693?u=balupton and https://github.com/bevry/eslint-config-bevry would be useful as it detects the needs of the project and adapts the linting accordingly |
This was completed in #1905 |
Hello :) |
Eslint plugin deno, and VSCode extension could be amazing! |
The TypeScript team announced a couple months ago that they were focusing internally on migrating to ESLint from tslint and then earlier this week tslint mentioned that they were deprecating tslint and joining forces with ESLint.
Therefore I think it only makes sense we migrate to ESLint. It would also be a good point to re-evaluate our linting rules and ensure that we extend our rules based on ignoring anything that Prettier handles (currently there are a couple conflicts between Prettier and tslint we have to add ignore comments for).
The text was updated successfully, but these errors were encountered: