-
Notifications
You must be signed in to change notification settings - Fork 538
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
removed redundant message specifying plugin mode #2122
Conversation
In the past we had fabric8.mode in Fabric8 Maven Plugin which used to work both on Kubernetes and OpenShift. Now we've specific plugins, so we should not specify Kubernetes/Openshift mode in log messages.
Eclipse JKube CI ReportStarted new GH workflow run for #2122 (2023-04-03T12:20:21Z) ⚙️ JKube E2E Tests (4596385614)
|
While working on issue #1768, I came across one more redundant "mode" message. I think it's okay to remove this message too. What's your opinion @rohanKanojia @sunix ? |
Codecov Report
@@ Coverage Diff @@
## master #2122 +/- ##
=========================================
Coverage 56.74% 56.74%
Complexity 4315 4315
=========================================
Files 478 478
Lines 20981 20980 -1
Branches 2791 2791
=========================================
Hits 11905 11905
+ Misses 7860 7859 -1
Partials 1216 1216
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
...lugin/plugin/src/main/java/org/eclipse/jkube/maven/plugin/mojo/build/AbstractDockerMojo.java
Show resolved
Hide resolved
Kudos, SonarCloud Quality Gate passed! |
Description
Fixes a chore remaining from #1279
As noted by @manusa and @rohanKanojia, In the past we had fabric8.mode in Fabric8 Maven Plugin which used to work both on Kubernetes and OpenShift. Now we've specific plugins, so we should not specify Kubernetes/Openshift mode in log messages.
Type of change
test, version modification, documentation, etc.)
Checklist