-
Notifications
You must be signed in to change notification settings - Fork 105
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
Renaming a file #334
Labels
Comments
Probably it should send workspace/didRenameFiles (https://microsoft.github.io/language-server-protocol/specifications/lsp/3.17/specification/#workspace_didRenameFiles) |
Does this library support this already? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I'm not entirely sure, but it doesn't seem like lsp4intellij appropriately handles the case where someone renames a class which has an associated file name.
It seems to send the
didChange
notifications to modify the text references in different files, but it won't be able to track new changes done in that file since thetextDocument
key is now changed. I'm not an LSP expert, but given the file was opened in IntelliJ, shouldn't this trigger adidClose
on the old file name, and adidOpen
on the new one?The text was updated successfully, but these errors were encountered: