-
Notifications
You must be signed in to change notification settings - Fork 82
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
Update dependencies #920
Update dependencies #920
Conversation
* Gradle 8.5 -> 8.6 * Kotlin 1.9.21 -> 1.9.22 * Ktor 2.3.7 -> 2.3.8 * kotlinx.coroutines 1.7.3 -> 1.8.0 * kotlinx.serialization 1.6.2 -> 1.6.3 * kotlin-logging 6.0.1 -> 6.0.3 * SLF4J 2.0.9 -> 2.0.12 * kotlin-node 18.16.12-pre.619 -> 20.11.5-pre.706 * kotlin-multiplatform-bignum 0.3.8 -> 0.3.9 * KSP 1.9.21-1.0.16 -> 1.9.22-1.0.17 * KotlinPoet 1.15.3 -> 1.16.0 * JUnit 5 5.10.1 -> 5.10.2 * MockK 1.13.8 -> 1.13.10 * AtomicFU 0.23.1 -> 0.23.2 * Binary compatibility validator 0.13.2 -> 0.14.0 * gradle-buildconfig-plugin 5.1.0 -> 5.3.5 * Foojay Toolchains Plugin 0.7.0 -> 0.8.0 * gradle/wrapper-validation-action v1 -> v2 * Replace gradle/gradle-build-action with gradle/actions/setup-gradle
I get this compiler error when trying to access
|
Downgrading kotlin-node usually helps |
hm, but i don't want to never update this dependency |
i've opened an issue, let's see what the response to that is: JetBrains/kotlin-wrappers#2221 |
the solution is to change the target JS module system from UMD to CommonJS / ES modules: 5bcc68e (see JetBrains/kotlin-wrappers#2221). i've chosen CommonJS as that's supported for all Node.js versions afaik, ES modules were supported at some later point. this also allows us to remove |
useCommonJs()
to fix issues with kotlin-node, see Compilation error when usingnode.process.process
JetBrains/kotlin-wrappers#2221