chore: Configure detekt Gradle task to show its result in the termina…

…l/IDE directly instead of generating an XML report (#1989)
JetBrains · Feb 13, 2024 · a67549c · a67549c
1 parent b909d07
commit a67549c
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 26 deletions.
diff --git a/build.gradle.kts b/build.gradle.kts
@@ -1,6 +1,6 @@
-import io.gitlab.arturbosch.detekt.Detekt
-import io.gitlab.arturbosch.detekt.report.ReportMergeTask
-import org.jetbrains.exposed.gradle.*
+import org.jetbrains.exposed.gradle.configureDetekt
+import org.jetbrains.exposed.gradle.configurePublishing
+import org.jetbrains.exposed.gradle.testDb
 
 plugins {
     kotlin("jvm") apply true
@@ -15,8 +15,6 @@ repositories {
 }
 
 allprojects {
-    configureDetekt()
-
     if (this.name != "exposed-tests" && this.name != "exposed-bom" && this != rootProject) {
         configurePublishing()
     }
@@ -26,20 +24,12 @@ apiValidation {
     ignoredProjects.addAll(listOf("exposed-tests", "exposed-bom"))
 }
 
-val reportMerge by tasks.registering(ReportMergeTask::class) {
-    output.set(rootProject.buildDir.resolve("reports/detekt/exposed.xml"))
-}
-
 subprojects {
+    configureDetekt()
+
     dependencies {
         detektPlugins(rootProject.libs.detekt.formatting)
     }
-    tasks.withType<Detekt>().configureEach detekt@{
-        finalizedBy(reportMerge)
-        reportMerge.configure {
-            input.from(this@detekt.xmlReportFile)
-        }
-    }
 }
 
 subprojects {

diff --git a/buildSrc/src/main/kotlin/org/jetbrains/exposed/gradle/Detekt.kt b/buildSrc/src/main/kotlin/org/jetbrains/exposed/gradle/Detekt.kt
@@ -1,12 +1,10 @@
 package org.jetbrains.exposed.gradle
 
-import io.gitlab.arturbosch.detekt.Detekt
 import io.gitlab.arturbosch.detekt.DetektPlugin
 import io.gitlab.arturbosch.detekt.extensions.DetektExtension
 import org.gradle.api.Project
 import org.gradle.kotlin.dsl.apply
 import org.gradle.kotlin.dsl.configure
-import org.gradle.kotlin.dsl.withType
 
 fun Project.configureDetekt() {
     apply<DetektPlugin>()
@@ -17,12 +15,4 @@ fun Project.configureDetekt() {
         parallel = true
         config.setFrom("$rootDir/detekt/detekt-config.yml")
     }
-    tasks.withType<Detekt>().configureEach {
-        reports {
-            xml.required.set(true)
-            html.required.set(false)
-            txt.required.set(false)
-            sarif.required.set(false)
-        }
-    }
 }
diff --git a/detekt/detekt-config.yml b/detekt/detekt-config.yml
@@ -1,7 +1,7 @@
 # https://github.com/detekt/detekt/blob/main/detekt-core/src/main/resources/default-detekt-config.yml
 
 build:
-    maxIssues: 52
+    maxIssues: 0
 
 formatting:
     ArgumentListWrapping: