-
Notifications
You must be signed in to change notification settings - Fork 676
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
Implement completion for package references in csproj files #1156
Comments
This will likely happen at some point, but not for awhile (unless somebody wants to contribute! 😄) |
Hi, Are you planning on merging PR #224? In that case I should make the implementation for csproj somehow fit into what was there for project.json. Otherwise I'd use the project.json implementation as a template for csproj wherever it makes sense. I'd start by raising the PR asap so you can easily follow up on progress. Since you tagged this issue as Backlog, should I target Let me know your plans regarding #224 so I can compile the task list for my PR. Cheers, |
You're welcome to take a stab at it! It'll be quite some time before I'm able to get to this. |
Indeed editing without the intellisense is a bit of a pain so I've been using the dotnet cli to manipulate nuget packages. Though I would use this should it be completed. Good luck! |
BTW, there's a new extension for Visual Studio 2017 that provides https://github.com/dotnet/ProjFileTools Might help if anybody is curious about the mechanics of how to accomplish this. |
@mlorbetske and I were talking about how we might pull this into OmniSharp and C# for VS Code just yesterday. |
Go go @mlorbetske :-) |
Unfortunately, this is going to have to move to 1.11. I've started looking at it, but other responsibilities got in the way this month. |
Fortunately, 1.10 is a great release with lots of improvements. 👍🏻 |
Very waiting the enhancement. |
Hi. It's not part of OmniSharp, but I finally bit the bullet and wrote an extension to provide basic completion for BTW, I'm happy to contribute this functionality back to OmniSharp if it makes sense to do so. |
The current changelog for C# extension reads "Known Issues in 1.12.0...There currently is no completion support for package references in csproj files. (#1156)" Might be a good idea to add a link to @tintoy 's extension to tide people over in the interim. Or just simply adopt that extension wholesale as the official support. |
That's a very good idea (and thanks for bumping this issue to the top of my stack 😄). @tintoy: I'd definitely work with you to contribute this back into C# for VS Code if you're interested. In the meantime, I'll add some text to point folks to your extension. |
@DustinCampbell sure, I'm keen :) |
Bonus -- @tintoy just added ability to toggle filtering of pre-release versions! |
@DustinCampbell @tintoy Any update on this? It would be great to have this functionality integrated into C# for VS Code. |
@BryanHitchcock It's been a while since I looked into this, but I'm still open to the idea of doing this (and am happy to put in the effort if it's what users want and the OmniSharp team think it makes sense to do so 😄). I suspect this is mostly a deployment issue, rather than one of functionality (although I could be wrong) so it might be enough to ensure that MSBuild Project Tools gets installed when you install the C# extension? We previously discussed creating an extension pack but I don't think anyone's had a proper think about what that would look like so far. |
Hi, do you have any progress? Fixing Date? |
Whats the current state of things? There seems to be no alternatives. |
Environment data
dotnet --info
output:VS Code version:
1.8.1 ee428b0eead68bf0fb99ab5fdc4439be227b6281
C# Extension version: 1.6.2
Steps to reproduce
Edit a netcore csproj
Expected behavior
Intellisense when trying to get nuget package names and versions. This is currently possible in project.json. Editing this file manually without intellisense is a bit painful.
Actual behavior
No behaviour its just editing a text file
The text was updated successfully, but these errors were encountered: