Skip to content

Conversation

@pm47
Copy link
Member

@pm47 pm47 commented Oct 2, 2025

Let's take advantage of the upcoming major release to upgrade kotlin and other dependencies.

The main point of attention is a breaking change in kotlinx-datetime: see https://github.com/Kotlin/kotlinx-datetime?tab=readme-ov-file#deprecation-of-instant.

Let's take advantage of the upcoming major release to upgrade kotlin and other dependencies.

The main point of attention is a breaking change in `kotlinx-datetime`: see https://github.com/Kotlin/kotlinx-datetime?tab=readme-ov-file#deprecation-of-instant.
@pm47 pm47 requested review from dpad85 and t-bast October 2, 2025 08:56
api("fr.acinq.bitcoin:bitcoin-kmp:${libs.versions.bitcoinkmp.get()}")
api("org.jetbrains.kotlinx:kotlinx-coroutines-core:${libs.versions.kotlinx.coroutines.get()}")
api("org.jetbrains.kotlinx:kotlinx-serialization-core:${libs.versions.kotlinx.serialization.get()}")
api("org.jetbrains.kotlinx:kotlinx-serialization-cbor:${libs.versions.kotlinx.serialization.get()}")
Copy link
Member Author

Choose a reason for hiding this comment

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

@dpad85 This will need to be added to phoenix build.gradle.kts.

Copy link
Member

@t-bast t-bast left a comment

Choose a reason for hiding this comment

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

LGTM since tests seem to pass, but running Phoenix with that updated version is the best test to figure out if there are runtime issues.

@pm47 pm47 merged commit fa9dc06 into master Oct 3, 2025
2 checks passed
@pm47 pm47 deleted the kotlin-2.2.20 branch October 3, 2025 09:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants