Disable uploading by default

Author: runningcode

plugin-build/build.gradle.kts

@@ -1,4 +1,3 @@
-import com.github.jengelman.gradle.plugins.shadow.tasks.ShadowJar
 import io.sentry.android.gradle.internal.ASMifyTask
 import io.sentry.android.gradle.internal.BootstrapAndroidSdk
 import java.io.FileInputStream

plugin-build/src/main/kotlin/io/sentry/android/gradle/AndroidComponentsConfig.kt

@@ -218,18 +218,16 @@ fun AndroidComponentsExtension<*, *, *>.configure(
           )
           .toTransform(SingleArtifact.MERGED_MANIFEST)
       }
-    }
-    if (extension.sizeAnalysis.enabled.get() == true) {
-      val sentryTelemetryProvider =
-        variant.configureTelemetry(project, extension, cliExecutable, sentryOrg, buildEvents)
-      variant.configureUploadAppTasks(
-        project,
-        extension,
-        sentryTelemetryProvider,
-        cliExecutable,
-        sentryOrg,
-        sentryProject,
-      )
+      if (extension.sizeAnalysis.enabled.get() == true) {
+        variant.configureUploadAppTasks(
+          project,
+          extension,
+          sentryTelemetryProvider,
+          cliExecutable,
+          sentryOrg,
+          sentryProject,
+        )
+      }
     }
   }
 }
@@ -365,11 +363,11 @@ private fun Variant.configureProguardMappingsTasks(
         releaseInfo = releaseInfo,
       )
 
-      generateUuidTask.hookWithMinifyTasks(
-        project,
-        name,
-        dexguardEnabled && GroovyCompat.isDexguardEnabledForVariant(project, name),
-      )
+    generateUuidTask.hookWithMinifyTasks(
+      project,
+      name,
+      dexguardEnabled && GroovyCompat.isDexguardEnabledForVariant(project, name),
+    )
 
     uploadMappingsTask.hookWithAssembleTasks(project, variant)
 
@@ -380,7 +378,8 @@ private fun Variant.configureProguardMappingsTasks(
 }
 
 /**
- * Configure the upload AAB and APK tasks and set them up as finalizers on the respective producer tasks
+ * Configure the upload AAB and APK tasks and set them up as finalizers on the respective producer
+ * tasks
  */
 fun Variant.configureUploadAppTasks(
   project: Project,
@@ -408,7 +407,8 @@ fun Variant.configureUploadAppTasks(
       sentryProperties = sentryProps,
       taskSuffix = name.capitalized,
     )
-  // TODO we can use the listToArtifacts API in AGP 8.3+ https://github.com/android/gradle-recipes/tree/agp-8.10/listenToArtifacts
+  // TODO we can use the listToArtifacts API in AGP 8.3+
+  // https://github.com/android/gradle-recipes/tree/agp-8.10/listenToArtifacts
   project.afterEvaluate {
     getBundleTask(project, variant.name)!!.configure { it.finalizedBy(uploadBundleTask) }
     getAssembleTaskProvider(project, variant)!!.configure { it.finalizedBy(uploadApkTask) }

plugin-build/src/main/kotlin/io/sentry/android/gradle/extensions/SizeAnalysisExtension.kt

@@ -11,5 +11,7 @@ open class SizeAnalysisExtension
 constructor(objects: ObjectFactory, providerFactory: ProviderFactory) {
 
   val enabled: Property<Boolean> =
-    objects.property(Boolean::class.java).convention(providerFactory.isCi())
+    objects
+      .property(Boolean::class.java)
+      .convention(providerFactory.isCi() && false) // set to false for now otherwise upload fails CI
 }