From 07e78aa3b0bf9b6e6a890826f252a3194b28514a Mon Sep 17 00:00:00 2001
From: Laimonas Turauskas <laimiux@gmail.com>
Date: Thu, 4 Jan 2024 11:30:02 -0500
Subject: [PATCH] Fix issues with artifact publishing.

---
 .github/workflows/build-workflow.yml |  2 +-
 build.gradle.kts                     | 12 ++++++++++++
 formula-android/build.gradle.kts     |  6 ++++--
 formula-lint/build.gradle.kts        |  9 ---------
 gradle/libs.versions.toml            |  2 +-
 5 files changed, 18 insertions(+), 13 deletions(-)

diff --git a/.github/workflows/build-workflow.yml b/.github/workflows/build-workflow.yml
index ce27b42e..bcc135d4 100644
--- a/.github/workflows/build-workflow.yml
+++ b/.github/workflows/build-workflow.yml
@@ -27,7 +27,7 @@ jobs:
       - name: Run Formula Android Instrumentation Tests
         run: ./gradlew :formula-android-tests:testRelease
       - name: Run Formula Lint Tests
-        run: ./gradlew :formula-lint:test
+        run: ./gradlew :formula-lint:build
       - name: Generate Jacoco Report
         run: ./gradlew jacocoTestReportMerged
       - name: Save Jacoco Report
diff --git a/build.gradle.kts b/build.gradle.kts
index 77398c20..b1213783 100644
--- a/build.gradle.kts
+++ b/build.gradle.kts
@@ -115,4 +115,16 @@ tasks.register("install") {
     dependsOn(publishTasks)
 }
 
+// We disable jacoco report when running buildTask
+val isBuild = gradle.startParameter.taskNames.any { it.contains("build") }
+if (isBuild) {
+    subprojects {
+        plugins.withType(JacocoPlugin::class) {
+            tasks.withType<JacocoReport>() {
+                enabled = false
+            }
+        }
+    }
+}
+
 apply(from = "gradle/merge-reports.gradle")
diff --git a/formula-android/build.gradle.kts b/formula-android/build.gradle.kts
index f93d228e..fd43786d 100644
--- a/formula-android/build.gradle.kts
+++ b/formula-android/build.gradle.kts
@@ -1,3 +1,7 @@
+import com.android.build.gradle.internal.coverage.JacocoReportTask
+import com.android.build.gradle.internal.lint.AndroidLintAnalysisTask
+import com.android.build.gradle.internal.lint.LintModelWriterTask
+
 plugins {
     id("com.android.library")
     id("kotlin-android")
@@ -28,8 +32,6 @@ task<Javadoc>("javadoc") {
     source(android.sourceSets["main"].java.getSourceFiles())
     classpath = project.files(
         android.bootClasspath.joinToString(separator = File.pathSeparator),
-        configurations.api,
-        configurations.implementation
     )
 }
 
diff --git a/formula-lint/build.gradle.kts b/formula-lint/build.gradle.kts
index 14622a42..3ce39833 100644
--- a/formula-lint/build.gradle.kts
+++ b/formula-lint/build.gradle.kts
@@ -22,12 +22,3 @@ dependencies {
   testImplementation(libs.lint.tests)
   testImplementation(libs.junit)
 }
-
-// Need to register direct task dependencies since jacocoTestReport is
-// accessing the files produced by those lint tasks
-plugins.withId("jacoco") {
-  tasks.named("jacocoTestReport") {
-    dependsOn(tasks.withType<AndroidLintAnalysisTask>())
-    dependsOn(tasks.withType<LintModelWriterTask>())
-  }
-}
diff --git a/gradle/libs.versions.toml b/gradle/libs.versions.toml
index 73ea4971..72ec223e 100644
--- a/gradle/libs.versions.toml
+++ b/gradle/libs.versions.toml
@@ -1,6 +1,6 @@
 [versions]
 android-gradle = "8.2.0"
-dokka-gradle = "1.4.32"
+dokka-gradle = "1.9.10"
 
 kotlin = "1.9.10"
 coroutines = "1.5.2"