Kotlin 1.9.20: new target hierarchy setup, opt in to 'expect'/'actual' classes (hoc081098#193)

🎉🎉🎉🎉

Author: hoc081098

build.gradle.kts

@@ -5,6 +5,7 @@ import org.gradle.api.tasks.testing.logging.TestExceptionFormat
 import org.gradle.api.tasks.testing.logging.TestLogEvent
 import org.jetbrains.dokka.gradle.DokkaTask
 import org.jetbrains.kotlin.gradle.dsl.JvmTarget
+import org.jetbrains.kotlin.gradle.dsl.KotlinCompile
 import org.jetbrains.kotlin.gradle.plugin.mpp.KotlinNativeTargetWithTests
 import org.jetbrains.kotlin.gradle.plugin.mpp.NativeBuildType
 import java.net.URL
@@ -60,22 +61,18 @@ kotlin {
       }
     }
     browser {
-      testTask(
-        Action {
-          useMocha {
-            timeout = "10s"
-          }
-        },
-      )
+      testTask {
+        useMocha {
+          timeout = "10s"
+        }
+      }
     }
     nodejs {
-      testTask(
-        Action {
-          useMocha {
-            timeout = "10s"
-          }
-        },
-      )
+      testTask {
+        useMocha {
+          timeout = "10s"
+        }
+      }
     }
   }
 
@@ -107,130 +104,28 @@ kotlin {
   androidNativeX64()
 
   sourceSets {
-    val commonMain by getting {
+    commonMain {
       dependencies {
         api("org.jetbrains.kotlinx:kotlinx-coroutines-core:$coroutinesVersion")
       }
     }
-    val commonTest by getting {
+    commonTest {
       dependencies {
         implementation(kotlin("test-common"))
         implementation(kotlin("test-annotations-common"))
         implementation("org.jetbrains.kotlinx:kotlinx-coroutines-test:$coroutinesVersion")
       }
     }
-    val jvmMain by getting {
-      dependsOn(commonMain)
-    }
-    val jvmTest by getting {
-      dependsOn(commonTest)
-
+    jvmTest {
       dependencies {
         implementation(kotlin("test-junit"))
       }
     }
-    val jsMain by getting {
-      dependsOn(commonMain)
-    }
-    val jsTest by getting {
+    jsTest {
       dependencies {
         implementation(kotlin("test-js"))
       }
     }
-
-    val nativeMain by creating {
-      dependsOn(commonMain)
-    }
-    val nativeTest by creating {
-      dependsOn(commonTest)
-    }
-
-    val darwinMain by creating {
-      dependsOn(nativeMain)
-    }
-    val darwinTest by creating {
-      dependsOn(nativeTest)
-    }
-
-    val linuxMain by creating {
-      dependsOn(nativeMain)
-    }
-    val linuxTest by creating {
-      dependsOn(nativeTest)
-    }
-
-    val pthreadAndroidMain by creating {
-      dependsOn(nativeMain)
-    }
-    val pthreadAndroidTest by creating {
-      dependsOn(nativeTest)
-    }
-
-    val appleTargets =
-      listOf(
-        "iosX64",
-        "iosSimulatorArm64",
-        "iosArm64",
-        "macosX64",
-        "macosArm64",
-        "tvosArm64",
-        "tvosX64",
-        "tvosSimulatorArm64",
-        "watchosArm32",
-        "watchosArm64",
-        "watchosSimulatorArm64",
-        "watchosX64",
-        "watchosSimulatorArm64",
-        "watchosDeviceArm64",
-      )
-
-    val linuxTargets =
-      listOf(
-        "linuxX64",
-        "linuxArm64",
-      )
-
-    val androidNativeTargets =
-      listOf(
-        "androidNativeArm32",
-        "androidNativeArm64",
-        "androidNativeX86",
-        "androidNativeX64",
-      )
-
-    appleTargets.forEach {
-      getByName("${it}Main") {
-        dependsOn(darwinMain)
-      }
-      getByName("${it}Test") {
-        dependsOn(darwinTest)
-      }
-    }
-
-    linuxTargets.forEach {
-      getByName("${it}Main") {
-        dependsOn(linuxMain)
-      }
-      getByName("${it}Test") {
-        dependsOn(linuxTest)
-      }
-    }
-
-    androidNativeTargets.forEach {
-      getByName("${it}Main") {
-        dependsOn(pthreadAndroidMain)
-      }
-      getByName("${it}Test") {
-        dependsOn(pthreadAndroidTest)
-      }
-    }
-
-    getByName("mingwX64Main") {
-      dependsOn(nativeMain)
-    }
-    getByName("mingwX64Test") {
-      dependsOn(nativeTest)
-    }
   }
 
   // enable running ios tests on a background thread as well
@@ -250,6 +145,19 @@ kotlin {
   }
 }
 
+tasks.withType<KotlinCompile<*>>().configureEach {
+  kotlinOptions {
+    // 'expect'/'actual' classes (including interfaces, objects, annotations, enums,
+    // and 'actual' typealiases) are in Beta.
+    // You can use -Xexpect-actual-classes flag to suppress this warning.
+    // Also see: https://youtrack.jetbrains.com/issue/KT-61573
+    freeCompilerArgs +=
+      listOf(
+        "-Xexpect-actual-classes",
+      )
+  }
+}
+
 spotless {
   kotlin {
     target("**/*.kt")

src/pthreadAndroidMain/kotlin/com/hoc081098/flowext/internal/Lock.kt renamed to src/androidNativeMain/kotlin/com/hoc081098/flowext/internal/Lock.kt

@@ -42,6 +42,7 @@ import kotlinx.coroutines.flow.take
 
 private data class MyTuple2<A, B>(val first: A, val second: B)
 
+@Suppress("NOTHING_TO_INLINE")
 private inline infix fun <A, B> A.with(second: B) = MyTuple2(this, second)
 
 @ExperimentalCoroutinesApi

src/darwinMain/kotlin/com/hoc081098/flowext/internal/Lock.kt renamed to src/appleMain/kotlin/com/hoc081098/flowext/internal/Lock.kt

@@ -26,7 +26,6 @@ package com.hoc081098.flowext
 
 import com.hoc081098.flowext.utils.BaseTest
 import com.hoc081098.flowext.utils.test
-import kotlin.native.concurrent.SharedImmutable
 import kotlin.test.Test
 import kotlin.test.assertEquals
 import kotlinx.coroutines.ExperimentalCoroutinesApi
@@ -73,10 +72,8 @@ private fun <T, R> Flow<T>.scanSkipFirst(
   operation: suspend (acc: R, value: T) -> R,
 ): Flow<R> = scan(initial, operation).drop(1)
 
-@SharedImmutable
 private val zeroToTen = List(10) { it.toString() }
 
-@SharedImmutable
 private val reducer: (acc: State, value: Int) -> State = { state, action ->
   when (action) {
     // items