-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Update to Kotlin 1.6.0 #20714
Update to Kotlin 1.6.0 #20714
Conversation
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.
Nice to see some movement on this front. 🙂
Btw, this PR now has some merge conflicts.
integration-tests/gradle/src/main/resources/basic-kotlin-application-project/build.gradle
Show resolved
Hide resolved
integration-tests/gradle/src/main/resources/multi-module-kotlin-project/web/build.gradle.kts
Show resolved
Hide resolved
@evanchooly RC2 is out |
I've rebased this and added RC2. |
This workflow status is outdated as a new workflow run has been triggered. |
Rebased again and updated to final 1.6.0. Will probably take a separate stab at those Java 18 problems in another PR. @gsmet this might be a nice (noteworthy?) addition to 2.5.0.Final. |
This workflow status is outdated as a new workflow run has been triggered. Failing Jobs - Building c9e53a4
Full information is available in the Build summary check run. Failures⚙️ JVM Tests - JDK 11 #- Failing: integration-tests/kafka-streams
📦 integration-tests/kafka-streams✖
|
As discussed on Zulip, let's make sure it still works with Java 18 EA and let's not backport it to 2.5. |
Done, verified locally. |
This is actually bumping to 1.6.0-RC for now. This is to start getting the pieces in place for the eventual release. I'm filing the PR now to make sure we get all the workarounds removed for Kotlin on Java 16. It obviously can't be merged until 1.6.0 is final.