Turn on explicit API mode (cashapp#1270)

jackCHH · Jan 30, 2024 · 809f3c1 · 809f3c1
1 parent c3353ef
commit 809f3c1
Show file tree

Hide file tree

Showing 17 changed files with 144 additions and 135 deletions.
diff --git a/build.gradle b/build.gradle
@@ -1,3 +1,5 @@
+import org.jetbrains.kotlin.gradle.dsl.KotlinTopLevelExtension
+
 apply plugin: 'com.github.ben-manes.versions'
 
 buildscript {
@@ -62,6 +64,10 @@ subprojects {
     }
   }
 
+  plugins.withId('org.jetbrains.kotlin.jvm') {
+    extensions.getByName('kotlin').explicitApi()
+  }
+
   plugins.withId('com.vanniktech.maven.publish') {
     publishing {
       repositories {

diff --git a/paparazzi-annotations/src/main/java/app/cash/paparazzi/annotations/Paparazzi.kt b/paparazzi-annotations/src/main/java/app/cash/paparazzi/annotations/Paparazzi.kt
@@ -2,4 +2,4 @@ package app.cash.paparazzi.annotations
 
 @Target(AnnotationTarget.FUNCTION)
 @Retention(AnnotationRetention.BINARY)
-annotation class Paparazzi
+public annotation class Paparazzi
diff --git a/paparazzi-gradle-plugin/src/main/java/app/cash/paparazzi/gradle/PaparazziPlugin.kt b/paparazzi-gradle-plugin/src/main/java/app/cash/paparazzi/gradle/PaparazziPlugin.kt
@@ -50,7 +50,7 @@ import org.jetbrains.kotlin.gradle.plugin.mpp.KotlinAndroidTarget
 import java.util.Locale
 
 @Suppress("unused")
-class PaparazziPlugin : Plugin<Project> {
+public class PaparazziPlugin : Plugin<Project> {
   override fun apply(project: Project) {
     val supportedPlugins = listOf("com.android.application", "com.android.library", "com.android.dynamic-feature")
     project.afterEvaluate {
@@ -241,9 +241,9 @@ class PaparazziPlugin : Plugin<Project> {
     }
   }
 
-  abstract class PaparazziTask : DefaultTask() {
+  public abstract class PaparazziTask : DefaultTask() {
     @Option(option = "tests", description = "Sets test class or method name to be included, '*' is supported.")
-    open fun setTestNameIncludePatterns(testNamePattern: List<String>): PaparazziTask {
+    public open fun setTestNameIncludePatterns(testNamePattern: List<String>): PaparazziTask {
       project.tasks.withType(Test::class.java).configureEach {
         it.setTestNameIncludePatterns(testNamePattern)
       }

diff --git a/paparazzi-gradle-plugin/src/main/java/app/cash/paparazzi/gradle/PrepareResourcesTask.kt b/paparazzi-gradle-plugin/src/main/java/app/cash/paparazzi/gradle/PrepareResourcesTask.kt
@@ -32,49 +32,49 @@ import org.gradle.api.tasks.PathSensitivity
 import org.gradle.api.tasks.TaskAction
 
 @CacheableTask
-abstract class PrepareResourcesTask : DefaultTask() {
+public abstract class PrepareResourcesTask : DefaultTask() {
   @get:Input
-  abstract val packageName: Property<String>
+  public abstract val packageName: Property<String>
 
   @get:Input
-  abstract val targetSdkVersion: Property<String>
+  public abstract val targetSdkVersion: Property<String>
 
   @get:Input
-  abstract val compileSdkVersion: Property<String>
+  public abstract val compileSdkVersion: Property<String>
 
   @get:Input
-  abstract val projectResourceDirs: ListProperty<String>
+  public abstract val projectResourceDirs: ListProperty<String>
 
   @get:Input
-  abstract val moduleResourceDirs: ListProperty<String>
+  public abstract val moduleResourceDirs: ListProperty<String>
 
   @get:InputFiles
   @get:PathSensitive(PathSensitivity.RELATIVE)
-  abstract val aarExplodedDirs: ConfigurableFileCollection
+  public abstract val aarExplodedDirs: ConfigurableFileCollection
 
   @get:Input
-  abstract val projectAssetDirs: ListProperty<String>
+  public abstract val projectAssetDirs: ListProperty<String>
 
   @get:InputFiles
   @get:PathSensitive(PathSensitivity.RELATIVE)
-  abstract val aarAssetDirs: ConfigurableFileCollection
+  public abstract val aarAssetDirs: ConfigurableFileCollection
 
   @get:Input
-  abstract val nonTransitiveRClassEnabled: Property<Boolean>
+  public abstract val nonTransitiveRClassEnabled: Property<Boolean>
 
   @get:InputFiles
   @get:PathSensitive(PathSensitivity.NONE)
-  abstract val artifactFiles: ConfigurableFileCollection
+  public abstract val artifactFiles: ConfigurableFileCollection
 
   @get:OutputFile
-  abstract val paparazziResources: RegularFileProperty
+  public abstract val paparazziResources: RegularFileProperty
 
   private val projectDirectory = project.layout.projectDirectory
 
   private val gradleUserHomeDirectory = projectDirectory.dir(project.gradle.gradleUserHomeDir.path)
 
   @TaskAction
-  fun writeResourcesFile() {
+  public fun writeResourcesFile() {
     val out = paparazziResources.get().asFile
     out.delete()
 

diff --git a/...-preview-processor/src/main/java/app/cash/paparazzi/preview/processor/PreviewProcessor.kt b/...-preview-processor/src/main/java/app/cash/paparazzi/preview/processor/PreviewProcessor.kt
@@ -6,11 +6,13 @@ import com.google.devtools.ksp.processing.SymbolProcessorEnvironment
 import com.google.devtools.ksp.processing.SymbolProcessorProvider
 import com.google.devtools.ksp.symbol.KSAnnotated
 
-class PreviewProcessorProvider : SymbolProcessorProvider {
-  override fun create(environment: SymbolProcessorEnvironment) = PreviewProcessor(environment)
+public class PreviewProcessorProvider : SymbolProcessorProvider {
+  override fun create(
+    environment: SymbolProcessorEnvironment
+  ): PreviewProcessor = PreviewProcessor(environment)
 }
 
-class PreviewProcessor(
+public class PreviewProcessor(
   private val environment: SymbolProcessorEnvironment
 ) : SymbolProcessor {
   override fun process(resolver: Resolver): List<KSAnnotated> = emptyList()