-
-
Notifications
You must be signed in to change notification settings - Fork 2.8k
Question: will there be a tsserver plugin? #254
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
Comments
Just so I'm understanding correctly - the language server plugin exists to:
Does it give you any other benefits at compile time? i.e. if I run |
|
Plus, integrates with the codefix suggestions, so autofix all problems or autofix on save works more seamlessly. |
@mohsen1 I was under the impression that this ticket was only about allowing plugins to modify the transpilation output, not the command line output? |
@MayhemYDG My understanding is that plugins are only for tsserver. They can provide errors, warnings and QuickFixes but when using the compiler command line tool ( |
Ah I see, that makes sense. Unfortunate that tsc doesn't use the plugins. |
Just for reference, I found some docs about that here: https://github.com/Microsoft/TypeScript/wiki/Writing-a-Language-Service-Plugin |
Hi guys. I'm loving tsserver and TypeScript's language service. So I've created a language service plugin for ESLint, https://github.com/Quramy/typescript-eslint-language-service . To use this plugin, {
"compilerOptions": {
"module": "commonjs",
"target": "es5",
"plugins": [
{
"name": "typescript-eslint-language-service"
}
]
}
} This plugin depends on only the following packages:
And this plugin gets I'm welcome for PRs or feedbacks ;) |
Thanks a lot for putting that together @Quramy! I can see a few areas for improvement:
For (1) and (2) if there are APIs/points of extension we don't currently provide from our packages, just let us know and we can look at providing them where it makes sense!
Thanks again for all your work on this, it's great to have different choices available for IDE setups! |
@JamesHenry Thanks for your feedback. You’re right. For now, I use some internal APIs and duplicate some codes to directly convert from ts AST(not text) to ESLint AST. I want some public APIs so I’ll report them later. About (3), using typescript-estree is temporary. If the above duplication is removed, I can remove the dependency. |
@Quramy amazing! Thank you. Usage with Sublime Text LSP is flawless. |
I'm going to close this as it's not something we're going to look to support ourselves now or in the near future. Quramy has done a great job and has an existing solution that people can use if they wish. |
I was just wondering…
How does the endorsement of typescript-eslint in favor of tslint effect the usecase for https://github.com/Microsoft/typescript-tslint-plugin ?
Is there something similar planned for eslint?
I just saw this also mentioned here: #243 (comment)
The text was updated successfully, but these errors were encountered: