Skip to content
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

Merged
merged 4 commits into from
Nov 13, 2018

Conversation

testforstephen
Copy link
Contributor

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:

  • resolve the existing source attachment for the class file
  • attach the missing source jar to the class file

Signed-off-by: Jinbo Wang <jinbwan@microsoft.com>
@eclipse-ls-bot
Copy link
Contributor

Can one of the admins verify this patch?

Signed-off-by: Jinbo Wang <jinbwan@microsoft.com>
Signed-off-by: Jinbo Wang <jinbwan@microsoft.com>
@fbricon
Copy link
Contributor

fbricon commented Oct 30, 2018

@testforstephen can you also open a PR to expose that command on the vscode-java side?

Signed-off-by: Jinbo Wang <jinbwan@microsoft.com>
@testforstephen
Copy link
Contributor Author

@fbricon Peer PR on vscode-java side. redhat-developer/vscode-java#699

@fbricon fbricon merged commit 0e04c87 into eclipse-jdtls:master Nov 13, 2018
@fbricon
Copy link
Contributor

fbricon commented Nov 13, 2018

Thanks @testforstephen, that should be helpful!

@fbricon
Copy link
Contributor

fbricon commented Nov 13, 2018

@testforstephen can you please document the new command in the wiki?

@testforstephen testforstephen deleted the jinbo_sourceattachment branch November 14, 2018 01:53
@testforstephen
Copy link
Contributor Author

@fbricon should i add the new command in the vscode-java readme? right?

@fbricon
Copy link
Contributor

fbricon commented Nov 14, 2018

@testforstephen yes please

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants