Deprecated constructor SymbolInformation changed to WorkspaceSymbol #1227
Conversation
There was a problem hiding this comment.
@anusreelakshmi934 All the changes in this PR relate to classes in the lsp4mp4ij package, which have been copied from IntelliJ Quarkus. We should verify whether Red Hat has addressed the use of deprecated methods in their current code. If not, it may be worth contributing a PR to their repository. Since we periodically update lsp4mp4ij from IntelliJ Quarkus, maintaining alignment with the Red Hat version is crucial from a maintenance perspective. Each update carries the risk of losing changes made locally in Liberty Tools for IntelliJ.
@dessina-devasia I checked the intellij quarks repo. They have not made the change. Its still the deprecated code. So Shall I go ahead and open a PR ? |
Sure @anusreelakshmi934. Please go ahead to open a PR in IntelliJ Quarkus |
|
Created an Issue and Opened a PR in IntelliJ Quarkus - redhat-developer/intellij-quarkus#1424 |
| import org.eclipse.lsp4j.Range; | ||
| import org.eclipse.lsp4j.SymbolInformation; | ||
| import org.eclipse.lsp4j.SymbolKind; | ||
| import org.eclipse.lsp4j.*; |
There was a problem hiding this comment.
It will be better to import required class only
There was a problem hiding this comment.
Check if the IDE made this change automatically.
There was a problem hiding this comment.
Yes, this import was automatically added by IntelliJ because more than three classes were being used in the code. As a result, it consolidated the imports.
Fixes #1186 #1182 #1181 #1180
The SymbolInformation Constructor is Changed to WorkspaceSymbol as per LSP4J documentation