-
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
Bump kotlin to 1.5.30, coroutines to 1.5.1, dokka to 1.5.0 #19632
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.
I wonder if .30 will fix all our Java 16+ issues with needing all the flags to open up modules. I'm on PTO until tomorrow but can take a look thursday.
No, that's what I tried to say with:
though I haven't checked whether fewer flags are needed. It still needs at least "java.util". This doesn't look so great either: https://youtrack.jetbrains.com/issue/KT-45545#focus=Comments-27-5141183.0-0 |
Can this be backported to 2.2 or is it easy to migrate to 2.3 (and does it come soon)? |
@MartinX3 I might find some time to check whether this would backport cleanly, but I cannot promise anyting. |
@MartinX3 I have been told that the next 2.2.x will take a while, so you might be better off updating to 2.3 when it's out. |
@famod Oh |
Fixes the two Gradle tests on Java 17 EA (#18372 (comment)).
I bumped the other two for general alignment. I ran dokka-maven-plugin via
-Ddokka
and it LGTM.PS: Both kapt and dokka (would) still need
add-opens
. Only that NoSuchMethodError should be gone now: https://youtrack.jetbrains.com/issue/KT-47583