-
Notifications
You must be signed in to change notification settings - Fork 8
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
Doesn't work with telescope #7
Comments
sorry for have view this issue for so long time. I will take a view recently.. sorry for that |
Is this when looking up class definitions for source-generated classes? I'm getting this issue too on Godot C# scripts when I try to go to a partial class. |
Emm.. I have no cludes to fix it.. can you provide the code in telescope? telescope doesn't come to the handler, and lspsaga do not have problem.. I see, it use another api.. I will do it |
I think it is telescope's problem it request data directly from neovim, do not from the handlers by nvim-lspconfig.. maybe, it never request from me |
find the code in neovim local function request_with_options(name, params, options)
local req_handler
if options then
req_handler = function(err, result, ctx, config)
local client = vim.lsp.get_client_by_id(ctx.client_id)
local handler = client.handlers[name] or vim.lsp.handlers[name]
handler(err, result, ctx, vim.tbl_extend('force', config or {}, options))
end
end
request(name, params, req_handler)
end all right, the pr is mime |
nvim-telescope/telescope.nvim#2770 it is the problem of telescope, and there is already a pr for it |
It seems the issue has to do with multiple values being returned when finding a C# partial class that has other source-generated classes. I created a temporary fix for "go to definition" that just chooses the first value returned here https://github.com/Spycemyster/dotnet-goto.nvim |
I have add support for telescope now, but a very basic one, I think I have solved it |
Now telescope is supported, please take a look at README.md. Thanks for your issue |
This plugin works correctly when we are not using Telescope to list definitions, but it throws an error when we try to use telescope
The text was updated successfully, but these errors were encountered: