use top-level config

quiltdata · Sep 3, 2024 · 07883ef · 07883ef
1 parent 35a7c17
commit 07883ef
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 66 deletions.
diff --git a/build.gradle b/build.gradle
@@ -90,6 +90,30 @@ allprojects {
         maven { url 'https://oss.sonatype.org/content/repositories/snapshots' }
     }
 
+    configurations {
+        // see https://docs.gradle.org/4.1/userguide/dependency_management.html#sub:exclude_transitive_dependencies
+        all*.exclude group: 'org.codehaus.groovy', module: 'groovy-all'
+        all*.exclude group: 'org.codehaus.groovy', module: 'groovy-cli-picocli'
+        // groovydoc libs
+        groovyDoc.extendsFrom runtime
+    }
+
+    dependencies {
+        // see https://docs.gradle.org/4.1/userguide/dependency_management.html#sec:module_replacement
+        implementation 'com.github.groovy-wslite:groovy-wslite:1.1.3'
+        implementation 'org.codehaus.groovy:groovy-all:3.0.22'
+        implementation 'com.fasterxml.jackson.core:jackson-databind:2.17.2'
+        implementation 'com.upplication:s3fs:2.2.2'
+
+        modules {
+            module('commons-logging:commons-logging') { replacedBy('org.slf4j:jcl-over-slf4j') }
+        }
+        // Documentation required libraries
+        groovyDoc 'org.fusesource.jansi:jansi:2.4.1'
+        groovyDoc 'org.codehaus.groovy:groovy-groovydoc:3.0.22'
+        groovyDoc 'org.codehaus.groovy:groovy-ant:3.0.22'
+    }
+
     test {
         useJUnitPlatform()
         finalizedBy jacocoTestReport // report is always generated after tests run
@@ -131,35 +155,6 @@ allprojects {
         }
     }
 
-
-    configurations {
-        // see https://docs.gradle.org/4.1/userguide/dependency_management.html#sub:exclude_transitive_dependencies
-        all*.exclude group: 'org.codehaus.groovy', module: 'groovy-all'
-        all*.exclude group: 'org.codehaus.groovy', module: 'groovy-cli-picocli'
-        // groovydoc libs
-        groovyDoc.extendsFrom runtime
-    }
-
-    dependencies {
-        // see https://docs.gradle.org/4.1/userguide/dependency_management.html#sec:module_replacement
-        implementation 'com.github.groovy-wslite:groovy-wslite:1.1.3'
-        implementation 'org.codehaus.groovy:groovy-all:3.0.22'
-        implementation 'com.fasterxml.jackson.core:jackson-databind:2.17.2'
-        implementation 'com.upplication:s3fs:2.2.2'
-
-        modules {
-            module('commons-logging:commons-logging') { replacedBy('org.slf4j:jcl-over-slf4j') }
-        }
-        // Documentation required libraries
-        groovyDoc 'org.fusesource.jansi:jansi:2.4.1'
-        groovyDoc 'org.codehaus.groovy:groovy-groovydoc:3.0.22'
-        groovyDoc 'org.codehaus.groovy:groovy-ant:3.0.22'
-    }
-
-    test {
-        useJUnitPlatform()
-    }
-
     // this is required due to this IDEA bug
     // https://youtrack.jetbrains.com/issue/IDEA-129282
     sourceSets {

diff --git a/plugins/nf-quilt/build.gradle b/plugins/nf-quilt/build.gradle
@@ -114,41 +114,4 @@ dependencies {
 // use JUnit 5 platform
 test {
     useJUnitPlatform()
-    finalizedBy jacocoTestReport // report is always generated after tests run
-}
-
-jacoco {
-    toolVersion = "0.8.12"
-    reportsDirectory = layout.buildDirectory.dir('reports/jacoco')
-}
-
-jacocoTestReport {
-    dependsOn test // tests are required to run before generating the report
-    reports {
-        xml.required = false
-        csv.required = false
-        html.outputLocation = layout.buildDirectory.dir('jacocoHtml')
-    }
-}
-
-jacocoTestCoverageVerification {
-    violationRules {
-        rule {
-            limit {
-                minimum = 0.5
-            }
-        }
-
-        rule {
-            enabled = false
-            element = 'CLASS'
-            includes = ['org.gradle.*']
-
-            limit {
-                counter = 'LINE'
-                value = 'TOTALCOUNT'
-                maximum = 0.3
-            }
-        }
-    }
 }