-
Notifications
You must be signed in to change notification settings - Fork 5.1k
chore: remove unused System.out|err #19525
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
base: main
Are you sure you want to change the base?
Conversation
🌟 Thank you for your contribution to the Apache Camel project! 🌟 🤖 CI automation will test this PR automatically. 🐫 Apache Camel Committers, please review the following items:
|
Likely the execution of this script is failing:
we may have included some change affecting the report generation. I'll have a look asap. |
Probably we also need to include logging configuration in those modules which are missing it. |
...chetype-component/src/main/resources/archetype-resources/src/main/java/__name__Producer.java
Outdated
Show resolved
Hide resolved
...camel-as2/camel-as2-api/src/test/java/org/apache/camel/component/as2/api/AS2MessageTest.java
Show resolved
Hide resolved
Will you change these issues or should I commit to your branch in your fork? |
Co-authored-by: Babak Vahdat <[email protected]>
…mel/component/as2/api/AS2MessageTest.java chore: remove log param Co-authored-by: Babak Vahdat <[email protected]>
d554a71
to
e605997
Compare
e605997
to
f0a3d17
Compare
It seems that we don't tolerate changes in archetypes as the CI is failing for some reason. I've reverted that change to continue with this PR and reported the problem to be fixed separately: CAMEL-22537 |
Description
Target
main
branch)Tracking
Apache Camel coding standards and style
mvn clean install -DskipTests
locally from root folder and I have committed all auto-generated changes.