-
Notifications
You must be signed in to change notification settings - Fork 413
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
Support resolve/update the source attachment for the binary classfile #837
Support resolve/update the source attachment for the binary classfile #837
Conversation
Signed-off-by: Jinbo Wang <jinbwan@microsoft.com>
Can one of the admins verify this patch? |
Signed-off-by: Jinbo Wang <jinbwan@microsoft.com>
Signed-off-by: Jinbo Wang <jinbwan@microsoft.com>
...lipse.jdt.ls.core/src/org/eclipse/jdt/ls/core/internal/commands/SourceAttachmentCommand.java
Outdated
Show resolved
Hide resolved
....jdt.ls.tests/src/org/eclipse/jdt/ls/core/internal/commands/SourceAttachmentCommandTest.java
Outdated
Show resolved
Hide resolved
@testforstephen can you also open a PR to expose that command on the vscode-java side? |
@fbricon Peer PR on vscode-java side. redhat-developer/vscode-java#699 |
Thanks @testforstephen, that should be helpful! |
@testforstephen can you please document the new command in the wiki? |
@fbricon should i add the new command in the vscode-java readme? right? |
@testforstephen yes please |
Signed-off-by: Jinbo Wang jinbwan@microsoft.com
This PR is to fix bug redhat-developer/vscode-java#233 (Add ability to attach missing sources). It provides two comamnds: