search

Hoffs / omnisharp-extended-lsp.nvim

Extended 'textDocument/definition' handler for OmniSharp Neovim LSP (now also `textDocument/references`, `textDocument/implementation` and source generated files)
147 stars 18 forks source link

Add windows support #4

Closed ilham-agustiawan closed 2 years ago

ilham-agustiawan commented 2 years ago

Go to definitions currently doesn't support windows. I think because the matcher and defolderize only catch / but not \\.

lsp.log

[DEBUG][2022-01-21 14:11:54] .../lua/vim/lsp.lua:1023   "LSP[omnisharp]"    "client.request"    1   "textDocument/definition"   {  position = {    character = 18,    line = 4  },  textDocument = {    uri = "file:///C:/Users/IlhamAgustiawan/dev/personal/TodoApi/Program.cs"  }}    <function 1>    3
[DEBUG][2022-01-21 14:11:54] .../vim/lsp/rpc.lua:347    "rpc.send"  {  id = 3,  jsonrpc = "2.0",  method = "textDocument/definition",  params = {    position = {      character = 18,      line = 4    },    textDocument = {      uri = "file:///C:/Users/IlhamAgustiawan/dev/personal/TodoApi/Program.cs"    }  }}
[DEBUG][2022-01-21 14:11:54] .../vim/lsp/rpc.lua:454    "rpc.receive"   {  id = 3,  jsonrpc = "2.0",  result = {    range = {      end = {        character = 48,        line = 42      },      start = {        character = 34,        line = 42      }    },    uri = "file:///%24metadata%24%5CProject%5CTodoApi%5CAssembly%5CMicrosoft%5CAspNetCore%5CMvc%5CSymbol%5CMicrosoft%5CExtensions%5CDependencyInjection%5CMvcServiceCollectionExtensions.cs"  }}
Hoffs commented 2 years ago

I believe it should be fixed with latest changes. Once you have time please test and confirm that it now works for you too.

ilham-agustiawan commented 2 years ago

Thanks. I tested in windows, it's working. Good job 👍