-
Notifications
You must be signed in to change notification settings - Fork 459
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
KubernetesWatchTask Prerequisite refactor in JKube Kit #1127
KubernetesWatchTask Prerequisite refactor in JKube Kit #1127
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1127 +/- ##
============================================
+ Coverage 49.24% 49.26% +0.02%
- Complexity 3608 3615 +7
============================================
Files 456 455 -1
Lines 20588 20595 +7
Branches 2802 2809 +7
============================================
+ Hits 10138 10146 +8
+ Misses 9379 9373 -6
- Partials 1071 1076 +5
Continue to review full report at Codecov.
|
fbe0d52
to
bbedf2b
Compare
.groupId(SPRING_BOOT_GROUP_ID) | ||
.artifactId(SPRING_BOOT_MAVEN_PLUGIN_ARTIFACT_ID) |
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.
Please, use a string here, if there's an error/typo in the constant it won't be revealed
.groupId(SPRING_BOOT_GROUP_ID) | ||
.artifactId(SPRING_BOOT_GRADLE_PLUGIN_ARTIFACT_ID) |
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.
bbedf2b
to
36e22ae
Compare
...s-maven-plugin/plugin/src/main/java/org/eclipse/jkube/maven/plugin/mojo/build/ApplyMojo.java
Show resolved
Hide resolved
...maven-plugin/plugin/src/main/java/org/eclipse/jkube/maven/plugin/mojo/develop/WatchMojo.java
Show resolved
Hide resolved
jkube-kit/common/src/main/java/org/eclipse/jkube/kit/common/util/JKubeProjectUtil.java
Show resolved
Hide resolved
.../plugin/src/test/java/org/eclipse/jkube/maven/plugin/mojo/GeneratedPluginDescriptorTest.java
Outdated
Show resolved
Hide resolved
+ SpringBootWatcher.isApplicable returns true for Spring Boot Gradle Plugin + Remove JKubeArtifactResolverService; Add utility method in JKubeProjectUtil to resolve artifact from dependencies instead + SpringBootUtil.getSpringBootPluginConfiguration also fetches configuration of Spring Boot Gradle Plugin Signed-off-by: Rohan Kumar <rohaan@redhat.com>
36e22ae
to
35e9d4a
Compare
SonarCloud Quality Gate failed. |
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, thx!
localRepositoryBaseDirectory only seemed to be only getting used by JKubeArtifactResolverService which was removed in eclipse-jkube#1127. Removing this field as it's no longer used anywhere. Signed-off-by: Rohan Kumar <rohaan@redhat.com>
localRepositoryBaseDirectory only seemed to be only getting used by JKubeArtifactResolverService which was removed in eclipse-jkube#1127. Removing this field as it's no longer used anywhere. Signed-off-by: Rohan Kumar <rohaan@redhat.com>
localRepositoryBaseDirectory only seemed to be only getting used by JKubeArtifactResolverService which was removed in eclipse-jkube#1127. Removing this field as it's no longer used anywhere. Signed-off-by: Rohan Kumar <rohaan@redhat.com>
localRepositoryBaseDirectory only seemed to be only getting used by JKubeArtifactResolverService which was removed in eclipse-jkube#1127. Removing this field as it's no longer used anywhere. Signed-off-by: Rohan Kumar <rohaan@redhat.com>
localRepositoryBaseDirectory only seemed to be only getting used by JKubeArtifactResolverService which was removed in #1127. Removing this field as it's no longer used anywhere. Signed-off-by: Rohan Kumar <rohaan@redhat.com>
Description
Prerequisite Refactor in JKube Kit for #1109
configuration of Spring Boot Gradle Plugin
Signed-off-by: Rohan Kumar rohaan@redhat.com
Type of change
test, version modification, documentation, etc.)
Checklist