-
Notifications
You must be signed in to change notification settings - Fork 3
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
Add a Language Server (LSP) #3
Comments
I agree, I think it would be good to have. |
So I'm actually gonna dig into this as part of the hackathon this week. You can watch my sanity slowly deteriorate using the |
You mean your sanity slowly become that on the level of |
Hmm, it looks like by just having a language server, you lose some features:
I've re-implemented word-based completion inside the language server, but the other features will also need to be addressed before releasing anything from this branch. |
Wait why is that so? Seems like just adding a language server provides a downgrade. Let's complain to the roslyn repository about it. |
It's due to the interaction between two (built-in) Visual Studio extensions. I was previously only using the TextMate one. It provides syntax highlighting, basic completion (snippets and word-based), and reference highlighting. When a language server is present, the LanguageServer extension takes over completion and reference highlighting (so it can call into the language server). Ideally, if a language server didn't declare these capabilities, it would fallback to the TextMate implementation, but it doesn't. (And I get why--it's hard to do.) It would also be nice if the TextMate snippets completion could coexist with the language server completion so it didn't have to re-implement it. |
Reopening per PR #27 |
I'll never have time for this, but it would be amazing if we could add a Language Server for T4 that delegates to the C# language server inside control blocks. (This is essentially how the Razor editor works.)
We'd map blocks of the T4 file to blocks of preprocessed C# class files to light up full C# support.
This would also enable full IntelliSense support for T4 Directive names and arguments.
Basic features
Advanced features
The text was updated successfully, but these errors were encountered: