Skip to content
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

Deprecate kotlin extension #6444

Closed
jamesnetherton opened this issue Sep 10, 2024 · 2 comments
Closed

Deprecate kotlin extension #6444

jamesnetherton opened this issue Sep 10, 2024 · 2 comments
Assignees
Milestone

Comments

@jamesnetherton
Copy link
Contributor

jamesnetherton commented Sep 10, 2024

Rethinking my comments in #6438. I think we should deprecate (for 3.15.0) and remove camel-quarkus-kotlin (in whatever release we first integrate Camel 4.9.0) along with the kotlin-dsl extension.

I don't see any major benefits to keeping camel-quarkus-kotlin. All it does is provide a Kotlin function wrapper around RouteBuilder.configure.

If folks want to build with Kotlin then it's simple enough to add quarkus-kotlin and write a RouteBuilder impl.

class Routes : RouteBuilder() {
    override fun configure() {
        from("timer:tick?period=5s")
            .log("Hello World")
    }
}
@jamesnetherton jamesnetherton self-assigned this Sep 10, 2024
jamesnetherton added a commit to jamesnetherton/camel-quarkus that referenced this issue Sep 10, 2024
jamesnetherton added a commit that referenced this issue Sep 10, 2024
@jamesnetherton jamesnetherton added this to the 3.15.0 milestone Sep 10, 2024
@apupier
Copy link
Contributor

apupier commented Sep 10, 2024

Camel Quarkus 3.15 will be based on Camel 4.7?

<camel.major.minor>4.7</camel.major.minor> <!-- run after each change: cd docs && mvnd validate -->

Camel Quarkus 3.16 will be based on Camel 4.9? (or 4.8)?

Double-checking that I understand well and we can remove the Kotlin DSL part with Camel 4.9.

@jamesnetherton
Copy link
Contributor Author

CQ 3.15.x will be LTS and will stay aligned to Camel 4.8.x
CQ 3.16.0 will probably be based on whatever the latest Camel 4.8.x release is. It depends if Camel 4.9.0 is available (unlikely TBH). So I guess we delay the extension removal until we integrate Camel 4.9.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants