You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Looks like we are a little over-aggressive here. Even if the client does not support workspace/applyEdit (which Zed does not), we can still just send code actions back with edit populated for the client to apply the edit locally.
The text was updated successfully, but these errors were encountered:
This alters the behaviour we initially implemented for #183 so the test that was added there will need to be updated accordingly. Looking at how things are now, we should have reviewed our implementations for textDocument/codeAction and textDocument/executeCommand when we made changes for #225.
dockerfile-language-server/src/server.ts
Lines 521 to 522 in 90dab55
Looks like we are a little over-aggressive here. Even if the client does not support
workspace/applyEdit
(which Zed does not), we can still just send code actions back withedit
populated for the client to apply the edit locally.The text was updated successfully, but these errors were encountered: