Skip to content

Implement support for workspace/symbol - #564

Open
niilohlin wants to merge 6 commits into
python-lsp:developfrom
niilohlin:workspace-symbols
Open

Implement support for workspace/symbol#564
niilohlin wants to merge 6 commits into
python-lsp:developfrom
niilohlin:workspace-symbols

Conversation

@niilohlin

@niilohlinniilohlin commented Jun 1, 2024

Copy link
Copy Markdown

solves #237 and #115 with inspiration from #511

Comment threadpylsp/python_lsp.py
@niilohlinniilohlin changed the title Implement workspace/symbolImplement support for workspace/symbolJun 6, 2024
@niilohlin
niilohlin marked this pull request as ready for review June 6, 2024 19:58
@feakuru

Copy link
Copy Markdown

you beat me to this and i could never make this so beautiful :D thanks for doing this!

@ibrokemypie

Copy link
Copy Markdown

What needs to be done for this to be merged?

@eyalk5

eyalk5 commented Apr 6, 2025

Copy link
Copy Markdown

Hi, it didn't work for me.
The good thing is that doing git merge is really easy.
I am not going to invest a lot debugging it.
can you write the exact configuration? I saw that it is enabled by default.

[DEBUG][2025-04-06 03:55:40] ...m/lsp/client.lua:678	"LSP[pylsp]"	"client.request"	1	"workspace/symbol"	{ query = ""}	<function 1>	5
[DEBUG][2025-04-06 03:55:40] .../vim/lsp/rpc.lua:286	"rpc.send"	{ id = 2, jsonrpc = "2.0", method = "workspace/symbol", params = { query = "" }}
[DEBUG][2025-04-06 03:55:40] ...m/lsp/client.lua:678	"LSP[pyright]"	"client.request"	2	"workspace/symbol"	{ query = ""}	<function 1>	5
[DEBUG][2025-04-06 03:55:40] .../vim/lsp/rpc.lua:286	"rpc.send"	{ id = 2, jsonrpc = "2.0", method = "workspace/symbol", params = { query = "" }}
[DEBUG][2025-04-06 03:55:40] .../vim/lsp/rpc.lua:408	"rpc.receive"	{ id = 2, jsonrpc = "2.0", result = {}}
[DEBUG][2025-04-06 03:55:40] .../vim/lsp/rpc.lua:408	"rpc.receive"	{ id = 2, jsonrpc = "2.0", result = {}}
``

@simuleite

Copy link
Copy Markdown

I need it !

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@niilohlin@feakuru@ibrokemypie@eyalk5@simuleite