-
Notifications
You must be signed in to change notification settings - Fork 399
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
Enhanced IBuildSupport usage to support other build tools such as bazel #1674
Conversation
Signed-off-by: Dmitry Bushenko <d.bushenko@gmail.com>
Can one of the admins verify this patch? |
org.eclipse.jdt.ls.core/src/org/eclipse/jdt/ls/core/internal/managers/IBuildSupport.java
Outdated
Show resolved
Hide resolved
@@ -77,16 +77,8 @@ public static boolean isJavaProject(IProject project) { | |||
return hasNature(project, JavaCore.NATURE_ID); | |||
} | |||
|
|||
public static boolean isMavenProject(IProject project) { |
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.
i will be cautious to remove public helpers from the utility since it may be used by other extensions.
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.
@testforstephen definitely cautious am I. The whole idea of improving IBuildSupport is to move the logic of working with particular build tool to the specific plugin. If we leave those methods in the utils class then we are leaving one more place for the maven/gradle logic. Which is bad, you know.
I understand that third-party plugins might rely on those methods. The will have to deal with the fact that they will need to improve their behavior and start relying on corresponding IBuildSupport.
Signed-off-by: Dmitry Bushenko <d.bushenko@gmail.com>
Signed-off-by: Dmitry Bushenko <d.bushenko@gmail.com>
Based on eclipse-jdtls#1674 Signed-off-by: Siarhei Leanavets <siarhei_leanavets1@epam.com>
superseded by #1694 |
Based on eclipse-jdtls#1674 Signed-off-by: Siarhei Leanavets <siarhei_leanavets1@epam.com>
Based on eclipse-jdtls#1674 Signed-off-by: Siarhei Leanavets <siarhei_leanavets1@epam.com>
Removed hardcoded checks for maven and gradle build tools.
Removed implicit assumption that there only two build tools: maven and gradle.
Enhanced IBuildSupport usage so that we may add more build tools such as bazel.
Signed-off-by: Dmitry Bushenko d.bushenko@gmail.com