-
Notifications
You must be signed in to change notification settings - Fork 23
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
Investigate adding compatibility for ESLint rules #517
Comments
Related news https://medium.com/palantir/tslint-in-2019-1a144c2317a9
Does the above mean that people that want to use ESLint will not be able to integrate their config into running I think for more adoption it would be a good idea to provide tooling similar to Would be happy about any explaination of perspectives on this topic. |
Palantir abandoned TSLint about 2 years ago. I expect nothing to come from their end regarding the topics in the blog post. Instead they will rely on other contributors to get things done.
Emulating the whole linter (cascading config, processors, .eslintignore, the actual linting, ...) is a bigger task than just loading and executing ESLint rules and displaying the results. This approach doesn't allow mixing rules from different linter engines in a single config (and therefore in a single execution of What I had in mind was a plugin like Of course there could be a plugin to completely emulate ESLint. The only real benefit to executing |
Thx for the clarification, sounds like a solid plan for me. |
Probably required to correctly convert the AST: typescript-eslint/typescript-eslint#774 |
The TypeScript team is moving towards ESLint because of its performance and obviously bigger ecosystem: microsoft/TypeScript#29288
This proposes a new plugin module that can load and execute ESLint rules.
Because of the completely different AST traversal it needs to be evaluated if that's even possible (without sacrificing performance too much).
This does not propose a plugin like
@fimbul/valtyr
that can completely emulate ESLint. This seems unnecessary as ESLint already works reasonably well on its own.The text was updated successfully, but these errors were encountered: