-
Notifications
You must be signed in to change notification settings - Fork 408
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
Gradle Project hasn't been updated properly #1617
Conversation
org.eclipse.jdt.ls.tests/projects/gradle/sample/app/build.gradle
Outdated
Show resolved
Hide resolved
org.eclipse.jdt.ls.core/src/org/eclipse/jdt/ls/core/internal/managers/GradleBuildSupport.java
Outdated
Show resolved
Hide resolved
@testforstephen I have updated the PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks fine to me. I'm able to reproduce, and confirm the changes fixes things, with the project structures referenced in the issues. I would just add a reference to one or both of the issues in the commit body.
Fixed. |
Updated change also works for me. Gradle project is detected and libraries are available as defined by the build file. |
org.eclipse.jdt.ls.core/src/org/eclipse/jdt/ls/core/internal/managers/BasicFileDetector.java
Outdated
Show resolved
Hide resolved
org.eclipse.jdt.ls.core/src/org/eclipse/jdt/ls/core/internal/managers/GradleBuildSupport.java
Show resolved
Hide resolved
- redhat-developer/vscode-java#1714 - redhat-developer/vscode-java#1463 Signed-off-by: Snjezana Peco <snjezana.peco@redhat.com>
@testforstephen I have updated the PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. thanks to @snjeza
Fixes redhat-developer/vscode-java#1714
Signed-off-by: Snjezana Peco snjezana.peco@redhat.com