-
Notifications
You must be signed in to change notification settings - Fork 465
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
feat: Configurable Dekorate integration #36
feat: Configurable Dekorate integration #36
Conversation
Codecov Report
@@ Coverage Diff @@
## master #36 +/- ##
============================================
+ Coverage 24.46% 24.47% +0.01%
- Complexity 1334 1336 +2
============================================
Files 326 327 +1
Lines 17603 17608 +5
Branches 2691 2692 +1
============================================
+ Hits 4307 4310 +3
- Misses 12805 12807 +2
Partials 491 491
Continue to review full report at Codecov.
|
e42974e
to
a8acd90
Compare
@@ -260,6 +262,10 @@ | |||
@Parameter(property = "jkube.resourceType") | |||
private ResourceFileType resourceFileType = yaml; | |||
|
|||
// When resource generation is delegated to Dekorate, should fabric8 resources be merged with Dekorate's |
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.
Shouldn't it be jkube here?
@@ -338,6 +344,15 @@ public void executeInternal() throws MojoExecutionException, MojoFailureExceptio | |||
if (skipResource) { | |||
return; | |||
} | |||
if (useDekorate(project) && mergeWithDekorate) { | |||
log.info("Dekorate detected, merging fabric8 and Dekorate resources"); |
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.
typo,
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 good to me
a8acd90
to
4fc4243
Compare
Once fabric8io/fabric8-maven-plugin#1767 gets approved->merged:
|
4fc4243
to
22cb6ed
Compare
All requested changes in fabric8io/fabric8-maven-plugin#1767 have been ported. |
Thanks, I approved it during previous review too 😆 . Shall we merge this then? |
Signed-off-by: Marc Nuri <marc@marcnuri.com>
22cb6ed
to
d38465f
Compare
SonarCloud Quality Gate failed.
|
Yes please, IT tests are green again after eclipse-jkube/jkube-integration-tests#9 |
Relates to: