Skip to content

Kotlin 2.2 #1286

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Kotlin 2.2 #1286

wants to merge 9 commits into from
+235 −135

Conversation

Jolanrensen
Copy link
Collaborator

@Jolanrensen Jolanrensen commented Jun 27, 2025 •
edited
Loading

Fixes #995

  • Bumps Kotlin and KSP to 2.2
  • Bumps kotlinpoet and dokka as well
  • Moving to dev.zacsweers.kctfork for build testing with ksp
  • Fixes for all plugins
  • Disabled all tests for the compiler plugin, because Remove the compiler plugin from this repo in favor /JetBrains/kotlin #1290
  • Bumps jupyter to 0.13.0-481-1 as it's the latest version that does not rely on a kotlin dev version atm

@Jolanrensen Jolanrensen added the infrastructure GitHub actions, Gradle Scripts, Writerside, etc. label Jun 27, 2025
@Jolanrensen Jolanrensen mentioned this pull request Jun 27, 2025
Closed
@Jolanrensen Jolanrensen force-pushed the kotlin-2.2 branch 6 times, most recently from 8b5f085 to 5d92e20 Compare July 1, 2025 11:36
@Jolanrensen Jolanrensen marked this pull request as ready for review July 1, 2025 14:23
zaleslaw
zaleslaw reviewed Jul 2, 2025
View reviewed changes
...ywords-generator/src/main/kotlin/org/jetbrains/dataframe/keywords/KeywordsGeneratorAction.kt
@@ -21,6 +22,7 @@ abstract class KeywordsGeneratorAction : WorkAction<KeywordsGeneratorAction.Para
private val taskPackageName = "org.jetbrains.kotlinx.dataframe.keywords"

override fun execute() {
println("Generating keywords using Kotlin compiler: ${KotlinCompilerVersion.getVersion()}")
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

println? Is it a debug stuff?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zaleslaw zaleslaw zaleslaw approved these changes

@koperagen koperagen Awaiting requested review from koperagen

@AndreiKingsley AndreiKingsley Awaiting requested review from AndreiKingsley

Assignees
No one assigned
Labels
infrastructure GitHub actions, Gradle Scripts, Writerside, etc.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Update to Kotlin 2.2+
2 participants
@Jolanrensen @zaleslaw