-
Notifications
You must be signed in to change notification settings - Fork 62
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
Support Kotlin 1.9.10 #394
Conversation
bd83486
to
907cacc
Compare
d2625f7
to
429a660
Compare
This reverts commit 232bd45.
@tschuchortdev please take a look and let me know if anything's missing, I'll be happy to update. Including klibs from the classpath is the biggest change required to support Kotlin/JS in newer releases. |
Not at all. This is great! And hope this can be merged/released 👍 |
tschuchortdev/kotlin-compile-testing#394 works as needed, but isn't released. This unblocks Poko from that release.
tschuchortdev/kotlin-compile-testing#394 works as needed, but isn't released. This unblocks Poko from that release.
* Fully support use of Kotlin EAP versions * Remove workaround for failing K2 toString on native target (KT-61616) * Remove explicit declarations workaround (KT-61627) * Use a local copy of Kotlin Compile Testing: tschuchortdev/kotlin-compile-testing#394 works as needed, but isn't released * Update version table
1.9.20 is in process of being released, and I spent more time updating this PR to be compatible, but ran into a couple of issues. I found this fork, which I plan to switch to, as it's already updated to Kotlin 1.9.20: https://github.com/ZacSweers/kotlin-compile-testing. The declared goal of the fork is to keep up to date with new Kotlin releases and Zac as some will know is a very active developer who already maintains a number of smaller Kotlin-related projects, so I have faith that the fork will continue to be maintained. I'll leave this open in case it's merged but I have no plans to update it for 1.9.20. |
Note - I have no plans to update this for Kotlin 1.9.20, see #394 (comment).
@BoD I hope you don't mind but I've forked your PR #388 as I've also spent time trying to upgrade to a more recent Kotlin version and looking into the failures. The missing piece is the last commit 7969038. I also rebased on #380 as that's where some of the initial changes came from.