FabricMC / fabric-language-kotlin

Fabric language module for Kotlin.
Apache License 2.0
299 stars 33 forks source link

Add Kotlinx Serialization to FLK #35

Closed ejektaflex closed 3 years ago

ejektaflex commented 3 years ago

Kotlinx Serialization is just as much an official Kotlin library as Coroutines is, so it would make sense to bundle it into FLK along with Stdlib, Reflect and Coroutines.

This also "fixes" #34. That actually seems to be a Loom issue, as Loom is not removing the Kotlin Jars when Gradle runs. However, that's a different issue and I should probably move it to the Loom repository.

ejektaflex commented 3 years ago

I can also confirm that this does indeed work correctly when used alongside a mod that is using Kotlinx Serialization. I tested the built jar out with an in-progress mod requiring it. 👍