Create languageSettings extension and remove warnings by adding the c…

…orrect imports to the right module.
thomaskioko · Nov 11, 2024 · 0a3b6a7 · 0a3b6a7
1 parent 0051e6c
commit 0a3b6a7
Show file tree

Hide file tree

Showing 12 changed files with 73 additions and 10 deletions.
diff --git a/...s/src/main/kotlin/com/thomaskioko/tvmaniac/plugins/KotlinMultiplatformConventionPlugin.kt b/...s/src/main/kotlin/com/thomaskioko/tvmaniac/plugins/KotlinMultiplatformConventionPlugin.kt
@@ -12,6 +12,7 @@ import org.gradle.kotlin.dsl.withType
 import org.jetbrains.kotlin.gradle.dsl.KotlinMultiplatformExtension
 import org.jetbrains.kotlin.gradle.plugin.KotlinPlatformType
 import org.jetbrains.kotlin.gradle.plugin.mpp.KotlinNativeTarget
+import org.jetbrains.kotlin.gradle.tasks.KotlinCompile
 
 class KotlinMultiplatformConventionPlugin : Plugin<Project> {
   override fun apply(target: Project) = with(target) {
@@ -44,18 +45,10 @@ class KotlinMultiplatformConventionPlugin : Plugin<Project> {
       sourceSets.all {
         languageSettings {
           listOf(
-            "androidx.paging.ExperimentalPagingApi",
             "kotlin.RequiresOptIn",
             "kotlin.experimental.ExperimentalObjCName",
             "kotlin.time.ExperimentalTime",
-            "kotlinx.cinterop.BetaInteropApi",
-            "kotlinx.cinterop.ExperimentalForeignApi",
-            "kotlinx.coroutines.DelicateCoroutinesApi",
             "kotlinx.coroutines.ExperimentalCoroutinesApi",
-            "kotlinx.coroutines.FlowPreview",
-            "kotlinx.coroutines.InternalCoroutinesApi",
-            "kotlinx.serialization.ExperimentalSerializationApi",
-            "org.mobilenativefoundation.store.store5.ExperimentalStoreApi",
           ).forEach { optIn(it) }
         }
       }
@@ -103,6 +96,16 @@ class KotlinMultiplatformConventionPlugin : Plugin<Project> {
   }
 }
 
+fun Project.addLanguageArgs(vararg args: String){
+  extensions.configure<KotlinMultiplatformExtension> {
+    sourceSets.all {
+      languageSettings {
+        args.forEach { optIn(it) }
+      }
+    }
+  }
+}
+
 fun Project.addKspDependencyForAllTargets(dependencyNotation: Any) = addKspDependencyForAllTargets("", dependencyNotation)
 private fun Project.addKspDependencyForAllTargets(
   configurationNameSuffix: String,

diff --git a/core/base/build.gradle.kts b/core/base/build.gradle.kts
@@ -1,3 +1,5 @@
+import com.thomaskioko.tvmaniac.plugins.addLanguageArgs
+
 plugins {
   alias(libs.plugins.tvmaniac.android.library)
   alias(libs.plugins.tvmaniac.multiplatform)
@@ -16,3 +18,7 @@ kotlin {
 }
 
 android { namespace = "com.thomaskioko.tvmaniac.core.base" }
+
+addLanguageArgs(
+  "kotlinx.coroutines.InternalCoroutinesApi",
+)
diff --git a/core/base/src/iosMain/kotlin/com/thomaskioko/tvmaniac/core/base/scope/NsQueueDispatcher.kt b/core/base/src/iosMain/kotlin/com/thomaskioko/tvmaniac/core/base/scope/NsQueueDispatcher.kt
@@ -5,7 +5,6 @@ import kotlinx.coroutines.CancellableContinuation
 import kotlinx.coroutines.CoroutineDispatcher
 import kotlinx.coroutines.Delay
 import kotlinx.coroutines.DisposableHandle
-import kotlinx.coroutines.InternalCoroutinesApi
 import kotlinx.coroutines.Runnable
 import platform.darwin.DISPATCH_TIME_NOW
 import platform.darwin.dispatch_after
@@ -16,7 +15,6 @@ import platform.darwin.dispatch_time
 
 val applicationNsQueueDispatcher: CoroutineDispatcher = NsQueueDispatcher(dispatch_get_main_queue())
 
-@InternalCoroutinesApi
 internal class NsQueueDispatcher(private val dispatchQueue: dispatch_queue_t) :
   CoroutineDispatcher(), Delay {
 

diff --git a/core/paging/build.gradle.kts b/core/paging/build.gradle.kts
@@ -1,3 +1,5 @@
+import com.thomaskioko.tvmaniac.plugins.addLanguageArgs
+
 plugins { id("plugin.tvmaniac.multiplatform") }
 
 kotlin {
@@ -10,3 +12,7 @@ kotlin {
     }
   }
 }
+
+addLanguageArgs(
+  "androidx.paging.ExperimentalPagingApi",
+)
diff --git a/core/util/build.gradle.kts b/core/util/build.gradle.kts
@@ -1,4 +1,5 @@
 import com.thomaskioko.tvmaniac.plugins.addKspDependencyForAllTargets
+import com.thomaskioko.tvmaniac.plugins.addLanguageArgs
 
 plugins {
   alias(libs.plugins.tvmaniac.android.library)
@@ -35,3 +36,9 @@ android {
 }
 
 addKspDependencyForAllTargets(libs.kotlinInject.compiler)
+
+addLanguageArgs(
+  "kotlinx.coroutines.ExperimentalCoroutinesApi",
+  "kotlinx.cinterop.ExperimentalForeignApi",
+  "kotlinx.cinterop.BetaInteropApi",
+)
diff --git a/data/popularshows/implementation/build.gradle.kts b/data/popularshows/implementation/build.gradle.kts
@@ -1,3 +1,5 @@
+import com.thomaskioko.tvmaniac.plugins.addLanguageArgs
+
 plugins {
   alias(libs.plugins.tvmaniac.multiplatform)
   alias(libs.plugins.ksp)
@@ -29,3 +31,7 @@ kotlin {
     commonTest { dependencies { implementation(libs.bundles.unittest) } }
   }
 }
+
+addLanguageArgs(
+  "androidx.paging.ExperimentalPagingApi",
+)
diff --git a/data/request-manager/api/build.gradle.kts b/data/request-manager/api/build.gradle.kts
@@ -1,3 +1,5 @@
+import com.thomaskioko.tvmaniac.plugins.addLanguageArgs
+
 plugins { alias(libs.plugins.tvmaniac.multiplatform) }
 
 kotlin {
@@ -10,3 +12,7 @@ kotlin {
     }
   }
 }
+
+addLanguageArgs(
+  "kotlinx.coroutines.ExperimentalCoroutinesApi",
+)
diff --git a/data/topratedshows/implementation/build.gradle.kts b/data/topratedshows/implementation/build.gradle.kts
@@ -1,3 +1,5 @@
+import com.thomaskioko.tvmaniac.plugins.addLanguageArgs
+
 plugins {
   alias(libs.plugins.tvmaniac.multiplatform)
   alias(libs.plugins.ksp)
@@ -26,3 +28,7 @@ kotlin {
     }
   }
 }
+
+addLanguageArgs(
+  "androidx.paging.ExperimentalPagingApi",
+)
diff --git a/data/trendingshows/implementation/build.gradle.kts b/data/trendingshows/implementation/build.gradle.kts
@@ -1,3 +1,5 @@
+import com.thomaskioko.tvmaniac.plugins.addLanguageArgs
+
 plugins {
   alias(libs.plugins.tvmaniac.multiplatform)
   alias(libs.plugins.ksp)
@@ -26,3 +28,7 @@ kotlin {
     }
   }
 }
+
+addLanguageArgs(
+  "androidx.paging.ExperimentalPagingApi",
+)
diff --git a/data/upcomingshows/implementation/build.gradle.kts b/data/upcomingshows/implementation/build.gradle.kts
@@ -1,3 +1,5 @@
+import com.thomaskioko.tvmaniac.plugins.addLanguageArgs
+
 plugins {
   alias(libs.plugins.tvmaniac.multiplatform)
   alias(libs.plugins.ksp)
@@ -26,3 +28,7 @@ kotlin {
     }
   }
 }
+
+addLanguageArgs(
+  "androidx.paging.ExperimentalPagingApi",
+)
diff --git a/datastore/implementation/build.gradle.kts b/datastore/implementation/build.gradle.kts
@@ -1,4 +1,5 @@
 import com.thomaskioko.tvmaniac.plugins.addKspDependencyForAllTargets
+import com.thomaskioko.tvmaniac.plugins.addLanguageArgs
 
 plugins {
   alias(libs.plugins.tvmaniac.android.library)
@@ -26,3 +27,9 @@ kotlin {
 addKspDependencyForAllTargets(libs.kotlinInject.compiler)
 
 android { namespace = "com.thomaskioko.tvmaniac.shared.domain.datastore.implementation" }
+
+addLanguageArgs(
+  "kotlinx.coroutines.ExperimentalCoroutinesApi",
+  "kotlinx.coroutines.InternalCoroutinesApi",
+  "kotlinx.cinterop.ExperimentalForeignApi"
+)
diff --git a/presentation/search/build.gradle.kts b/presentation/search/build.gradle.kts
@@ -1,3 +1,5 @@
+import com.thomaskioko.tvmaniac.plugins.addLanguageArgs
+
 plugins { id("plugin.tvmaniac.multiplatform") }
 
 kotlin {
@@ -33,3 +35,7 @@ kotlin {
     }
   }
 }
+
+addLanguageArgs(
+  "kotlinx.coroutines.ExperimentalCoroutinesApi",
+)