-
Notifications
You must be signed in to change notification settings - Fork 235
Pin Alpha versions of OmniSharp NuGet packages and adapt to new breaking changes #1299
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
Conversation
Clones removed
==============
+ src/PowerShellEditorServices/Services/TextDocument/Handlers/SignatureHelpHandler.cs -1
+ src/PowerShellEditorServices/Services/Workspace/Handlers/WorkspaceSymbolsHandler.cs -1
+ src/PowerShellEditorServices/Services/TextDocument/Handlers/HoverHandler.cs -1
+ src/PowerShellEditorServices/Services/TextDocument/Handlers/FormattingHandlers.cs -1
+ src/PowerShellEditorServices/Services/TextDocument/Handlers/CodeLensHandlers.cs -1
+ src/PowerShellEditorServices/Services/TextDocument/Handlers/FoldingRangeHandler.cs -2
+ src/PowerShellEditorServices/Services/TextDocument/Handlers/DocumentSymbolHandler.cs -1
+ src/PowerShellEditorServices/Services/TextDocument/Handlers/DefinitionHandler.cs -1
+ src/PowerShellEditorServices/Services/TextDocument/Handlers/CompletionHandler.cs -1
+ src/PowerShellEditorServices/Services/TextDocument/Handlers/DocumentHighlightHandler.cs -1
See the complete overview on Codacy |
Those releases are for us to try and fix: |
Just to double check we are on the same page: |
The Omnisharp folks don't usually release a "PR branch artifact". They only manually did it for me so that I could test out their fixes in: PowerShell/vscode-powershell#2627 I have a branch with all of the updated changes (all the breaking changes): and am planning on sending a PR with these changes in the next day or so (so long as PowerShell/vscode-powershell#2627 is fixed) I understand that the repo is in a bit of a bad state right now but PowerShell/vscode-powershell#2627 is such a bad bug that is affecting too many customers for me to focus on anything else at the moment. |
In other words - give me a day or two and things will (hopefully) be back to normal 😅 |
OK, gotcha, at first it looked like it was unintentional breakage. :-) |
I'm gonna close this PR. We should pin once the next stable release is out but for now, we need to use * |
The reason why the build is broken is because as per this comment
OmniSharp.Extensions.LanguageServer
andOmniSharp.Extensions.DebugAdapter.Server
instead of the master build artefact0.17.0-alpha*
, the build will still be broken because they made quite a few breaking changes between version0.17.0-alpha-0015
(your last green build) and0.17.0-alpha-0017
. Lots of namespace renames only though