-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
CAMEL-20888 - Remove Kotlin DSL #15481
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:
|
b861386
to
6a6724e
Compare
better to merge these other Pull Requests first to avoid breaking some other projects:
finally not needed, will be done later when projects will adopt Camel 4.9:
EDIT: only Camel Quarkus is required as Camel K is based on older version (only deprecation in Camel K would be nice) EDIT2: in fact, Camel Quarkus is still on Camel 4.7, James will take care of the upgrade when Camel Quarkus will move to Camel 4.9 |
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.
+1
6a6724e
to
2035a70
Compare
Need to regen as main branch has updated since |
2035a70
to
3de6fa5
Compare
@apupier the upgrade guide doc has a conflict |
Kotlin DSL was deprecated in Camel 4.7 It was not used a lot and it takes a fair part of the build time. Signed-off-by: Aurélien Pupier <[email protected]>
3de6fa5
to
5a44d0f
Compare
while checking camel-website if requring new changes, I found https://camel.apache.org/blog/2024/02/camel44-whatsnew/#kotlin-api
This PR is in fact removing all kotlin related things |
That is fine - all of this was experimental and manually hand code to be up to date. And there is little uptake of this. |
Kotlin DSL was deprecated in Camel 4.7
It was not used a lot and it takes a fair part of the build time.
Description
Target
camel-3.x
, whereas Camel 4 uses themain
branch)Tracking
Apache Camel coding standards and style
mvn clean install -DskipTests
locally from root folder and I have committed all auto-generated changes.