Strict plugin validation

slackhq · Oct 11, 2024 · a6aa53f · a6aa53f
1 parent 5eaa442
commit a6aa53f
Show file tree

Hide file tree

Showing 5 changed files with 10 additions and 0 deletions.
diff --git a/platforms/gradle/foundry-gradle-plugin/build.gradle.kts b/platforms/gradle/foundry-gradle-plugin/build.gradle.kts
@@ -55,6 +55,8 @@ tasks.named<ProcessResources>("processResources") {
 
 moshi { enableSealed.set(true) }
 
+tasks.named<ValidatePlugins>("validatePlugins") { enableStricterValidation.set(true) }
+
 // This is necessary for included builds, as the KGP plugin isn't applied in them and thus doesn't
 // apply disambiguation rules
 dependencies.constraints {

diff --git a/...dle/foundry-gradle-plugin/src/main/kotlin/foundry/gradle/dependencyrake/DependencyRake.kt b/...dle/foundry-gradle-plugin/src/main/kotlin/foundry/gradle/dependencyrake/DependencyRake.kt
@@ -48,6 +48,7 @@ import org.gradle.api.tasks.PathSensitivity
 import org.gradle.api.tasks.TaskAction
 import org.gradle.api.tasks.TaskProvider
 import org.gradle.api.tasks.UntrackedTask
+import org.gradle.work.DisableCachingByDefault
 
 private const val IGNORE_COMMENT = "// dependency-rake=ignore"
 
@@ -71,6 +72,7 @@ private val MANAGED_DEPENDENCIES =
  * advice to the project build file. This is usually not run directly, but rather added as a
  * finalizer to the `AdviceTask` it reads from.
  */
+@DisableCachingByDefault
 internal abstract class RakeDependencies
 @Inject
 constructor(objects: ObjectFactory, providers: ProviderFactory) : AbstractPostProcessingTask() {

diff --git a/.../gradle/foundry-gradle-plugin/src/main/kotlin/foundry/gradle/tasks/AndroidTestApksTask.kt b/.../gradle/foundry-gradle-plugin/src/main/kotlin/foundry/gradle/tasks/AndroidTestApksTask.kt
@@ -34,13 +34,15 @@ import org.gradle.api.tasks.PathSensitive
 import org.gradle.api.tasks.PathSensitivity.RELATIVE
 import org.gradle.api.tasks.TaskAction
 import org.gradle.api.tasks.TaskProvider
+import org.gradle.work.DisableCachingByDefault
 
 /**
  * A task that aggregates all the androidTest apk paths and writes them (newline-delimited) to an
  * [outputFile] in the format that Fladle expects.
  *
  * Not cacheable because this outputs absolute paths.
  */
+@DisableCachingByDefault
 public abstract class AndroidTestApksTask : DefaultTask() {
   @get:PathSensitive(RELATIVE)
   @get:InputFiles

diff --git a/...dle/foundry-gradle-plugin/src/main/kotlin/foundry/gradle/tasks/BaseDependencyCheckTask.kt b/...dle/foundry-gradle-plugin/src/main/kotlin/foundry/gradle/tasks/BaseDependencyCheckTask.kt
@@ -24,7 +24,9 @@ import org.gradle.api.provider.MapProperty
 import org.gradle.api.tasks.Input
 import org.gradle.api.tasks.TaskAction
 import org.gradle.internal.component.external.model.ModuleComponentArtifactIdentifier
+import org.gradle.work.DisableCachingByDefault
 
+@DisableCachingByDefault
 public abstract class BaseDependencyCheckTask : DefaultTask() {
   @get:Input public abstract val identifiersToVersions: MapProperty<String, String>
 

diff --git a/...rms/gradle/foundry-gradle-plugin/src/main/kotlin/foundry/gradle/tasks/BaseDownloadTask.kt b/...rms/gradle/foundry-gradle-plugin/src/main/kotlin/foundry/gradle/tasks/BaseDownloadTask.kt
@@ -29,6 +29,7 @@ import org.gradle.api.tasks.Input
 import org.gradle.api.tasks.OutputFile
 import org.gradle.api.tasks.TaskAction
 import org.gradle.internal.logging.progress.ProgressLoggerFactory
+import org.gradle.work.DisableCachingByDefault
 
 /**
  * Downloads a binary from its GitHub releases.
@@ -38,6 +39,7 @@ import org.gradle.internal.logging.progress.ProgressLoggerFactory
  *     ./gradlew <updateThing>
  * ```
  */
+@DisableCachingByDefault
 internal abstract class BaseDownloadTask(
   private val targetName: String,
   private val addExecPrefix: Boolean = false,