-
Notifications
You must be signed in to change notification settings - Fork 687
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
An update on the LSP-powered, Visual Studio Code C# Extension #5708
Comments
Thank you for this great decision! While I understand why Microsoft wants to keep closed-sourced such features as IntelliCode, I still do not understand why it was decided to move solution explorer and unit test features to closed-source extension. 😕 It looks not too unpleasant, but rather unclear, because this a very simple features to implement in comparison to LSP for example. Will you accept the community contribution which will bring that features to open-source extension? |
Really positive step forward. I think we would have liked a little more communication, rather than this surprise (to me at least) announcement. However, credit where credit is due - making this Open Source is a very good decision and gives me hope that C# can continue into the future in a strong position. This is such an important decision with the massive increase in choice and alternatives in the last few years. There is now hope that C# language support in VSCode can compete on an even footing. Thank you! |
This is great! After years of dealing with issues in VSCode, I had actually gave up last week and installed Rider. I'm very happy to see this announcement at the perfect time to give VSCode a second chance. Well done! Looking forward to trying it. |
Hallelujah, this is amazing. The dev kit seems to work really well on first usage, not personally fussed about that part being closed source. |
Thanks a lot for the great work. It's also amazing to see the C# Dev Kit announcement mentioning debugging, another key area that could use some love. I have a question if you may. Is IntellliCode supposed to work side by side with GitHub Copilot? In the past they seemed to do pretty much the same thing, so I kept only Copilot, but this seems different in the context of C# development. |
This is really good to hear! Well done! I honestly thought MS was going to regress, but this is a really great step forward! |
Great outcome. Time to update http://ghuntley.com/facture / https://isdotnetopen.com once the repo is open?
|
I will try it |
While I'm unsure what the license of |
Does anyone know how to customize formatting c# using the pre-release?
|
Have you tried the equivalent settings in |
Thank-you. That works very well. I added
|
cross projects renaming (method, class, etc) does not work yet |
Does it work with .NET 8 (i.e. preview) SDK projects? because a project targeting .NET8 was full of errors unlike .NET6/7 |
It seems there is no way to "unload" a project. e.g. a project that uses WinUI causes problem when loading the whole solution on macOS |
Please feel free to log an issue -- we can track specific issues easier |
|
I'm assuming you mean in this instance using the C# Dev Kit specifically? If so, please feel free to log an issue on this one. We've been discussing solution filters as well. For now what I've done is multiple .sln files (to serve as the 'filter') |
hey bro errors with new verssion |
That does not look like something we can help with :) |
no bro, i have 200 gb in disk space |
@NCLnclNCL Might want to report this over at dotnet/runtime. They can have someone who owns the installation experience look into this. We just call into the normal dotnet install script. thanks. :) |
dotnet-install.ps1 is in https://github.com/dotnet/install-scripts and that's the best place to open an issue for it. |
Will there be decompilation support like we had in Omnisharp? |
@timheuer the new language server to definition can't go to the source file. 2023-06-08.19.47.33.mov |
@TrieBr I suspect you might not have installed the extension yet, it's actually a new extension called "C# Dev Kit"... I had a similar issue initially |
@tmitchel2 , no, C# Dev Kit is not required for this features to work |
I had both C# and C# Dev Kit installed. I disabled them all and re-enabled and now it seems to be working! It seems my only remaining issue now is dotnet/razor#5901 Edit: This is such a drastic improvement. Great work! |
Nice work! |
I had multiple issues where when ctrl-clicking/F12 an item this would send me to the assembly file instead of the source file. Went back to Omnisharp until resolved |
I think you are hitting -- which we have a fix for coming soon: microsoft/vscode-dotnettools#40 |
Copy/paste files inside of the solution explorer isn't doing anything either. |
Can you please log an issue using the "Report an Issue" capability...this is related to the C# Dev Kit and not the language service. |
Nit pick: all the extension icons are 128x128 instead of 256x256. Appear blurred on retina display on the extension details page in VS Code. Not sure if that qualifies as a bug or a feature request :) |
Would you mind logging a bug so we can track that request -- we want to make sure to maximize your pixels ;-) |
For the folks who were having issues with go to definition not working, we've shipped an update this morning for the C# extension that should address those issues. Thanks for the reporst! |
@jasonmalinowski I still have the go to definition issue (mac os, devkit 0.1.103, net sdk 7.0.304). I don't see any errors in the output windows apart from
in the C# output window. Is there any suggested way to get better output / logging to float the issue? |
@tmitchel2 Ah, looks like you have something else going on then. Can you open another bug and we'll take up the conversation there? |
An existing issue is open here > microsoft/vscode-dotnettools#135. I've just realised that the new version you mentioned was unfortunately the source of my issues, I've downgraded and the issues have gone away. |
Is anyone else having trouble with windows defender? |
@jasonmalinowski I have same issue in form Also, there is questionable why all of this stuff download own copy of .NET 7.0.X runtime, is globally installed runtime is not enough? |
@tmitchel2: oh interesting, yes, that's not good! Could you file a new bug in this repo and we'll take the investigation from there? If you could include the full C# log if possible that'd be a help. |
Thanks everyone for the comments here on this update. This thread has started to turn in to some specific support/issues a bit, which we want to better accurately track for you and get the fix discussions happening in the right place. If you encounter issues with the C# Extension you can log them directly from within VS Code (Report an Issue) which will log directly on the repo or simply log issues directly on the repo at https://github.com/dotnet/vscode-csharp/issues. (Please note that the C# Dev Kit issues are tracked https://github.com/microsoft/vscode-dotnettools.) I'm closing this announcement issue for now. |
Last summer, the .NET team announced a plan to evolve the .NET Tooling ecosystem and incorporate more capabilities into Visual Studio Code. We appreciated your responses to that announcement, which were essential for us to determine the best path for improving the C# Extension and overall C# dev experience in VS Code. Building on that initial announcement and your feedback, today we released an update to the C# Extension that creates a more performant, reliable C# experience in VS Code by powering it with a new Language Server Protocol (LSP) server. This server communicates by default to Roslyn and Razor but allows users to choose an alternative language server. This updated, open-source extension is available today in the VS Code Marketplace exclusively as a pre-release version of the C# extension. The move of the repo and open sourcing of the extension code will take place in one week, on June 13. We are coordinating with a few individuals and teams to get the smoothest repo move possible.
Based on the community feedback from last year’s initial announcement, we decided to keep all components for this extension (including the LSP host) fully open source. We are committed to keeping the C# Extension and other components like Razor and Roslyn open source, and we are committed to further improving the quality and reliability of this extension over time.
As part of this effort, we also moved the C# for VS Code extension from github.com/OmniSharp/omnisharp-vscode to github.com/dotnet/vscode-csharp (in progress so not yet available). We will continue to monitor this new repo for feedback and provide regular updates to the extension until we are confident enough in the quality and feature capabilities to remove the Preview label. To get started using the new features, you must opt in to “Switch to Pre-release Version,”, which will install the new LSP-based extension.
You can also choose to revert from the LSP version to the OmniSharp version of the C# extension by first uninstalling the C# Dev Kit and then setting the
dotnet.server.useOmnisharp
to true. This will require a restart but will revert to using the original OmniSharp-powered extension. In the coming releases, the team is working to update the extension so that it can communicate with OmniSharp Server via LSP. Note that using the new C# Dev Kit extension requires the LSP-based extension to be running to work.Thanks for sharing your feedback so far! We look forward to continuing to improve and expand on your C# development experience in VS Code and Visual Studio.
The text was updated successfully, but these errors were encountered: