-
Notifications
You must be signed in to change notification settings - Fork 190
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
Publish rzls as a nuget package for vs-code #10453
Conversation
src/Razor/src/Microsoft.VisualStudio.DevKit.Razor/Microsoft.VisualStudio.DevKit.Razor.csproj
Show resolved
Hide resolved
src/Razor/src/Microsoft.VisualStudio.DevKit.Razor/Microsoft.VisualStudio.DevKit.Razor.csproj
Outdated
Show resolved
Hide resolved
I've validated locally with downloaded packages from this test insertion https://dev.azure.com/dnceng/internal/_build/results?buildId=2470510&view=artifacts&pathAsName=false&type=publishedArtifacts @DustinCampbell @davidwengier let me know if there's anything else blocking! Everything looks to be working for rzls with some changes in vscode (I'll link PR in a moment when I get it) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know if there will be fall-out from this for source build or VS Code, but I'm confident you'll fix any issues if there are 😁
I guess we'll see! |
There is some fallout in dotnet/sdk#41536 |
Fixes https://devdiv.visualstudio.com/DevDiv/_workitems/edit/2088486 Seems this was inadvertently removed in #10453 @ryzngard I'm pretty out of my depth here about what is required, but a local build of rzls with this change does fix the problem. Are there any other project properties that might be important, that I should bring back?
The majority of this was just copied from the way Roslyn packages their language server. Will do a test insertion to see if packages are actually published to vs-impl and can be consumed in vs-code before merging