Try to fix tag task dependency, ensure group is set for published art…

…ifacts
serpro69 · Sep 8, 2024 · ea7c374 · ea7c374
1 parent 9bb5d86
commit ea7c374
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 12 deletions.
diff --git a/build.gradle.kts b/build.gradle.kts
@@ -1,4 +1,5 @@
 import com.github.benmanes.gradle.versions.updates.DependencyUpdatesTask
+import io.github.serpro69.semverkt.gradle.plugin.tasks.TagTask
 import utils.configureGradleDaemonJvm
 
 plugins {
@@ -10,8 +11,6 @@ plugins {
 group = "io.github.serpro69"
 
 subprojects {
-    group = rootProject.group
-
     apply {
         plugin("com.github.ben-manes.versions")
     }
@@ -41,6 +40,14 @@ nexusPublishing {
     }
 }
 
+// Run :tag only after we've published artifacts to sonatype
+tasks.withType<TagTask>().configureEach {
+    // don't apply when "dryRun"
+    findProperty("dryRun") ?: run {
+        dependsOn("closeSonatypeStagingRepository")
+    }
+}
+
 apiValidation {
     ignoredProjects += listOf("bom", "cli-bot", /*"docs",*/ "test")
 }

diff --git a/buildSrc/src/main/kotlin/faker-kotlin-conventions.gradle.kts b/buildSrc/src/main/kotlin/faker-kotlin-conventions.gradle.kts
@@ -25,7 +25,6 @@ plugins {
     id("org.jetbrains.dokka")
 }
 
-group = rootProject.group.toString()
 val lib = project.libs
 
 dependencies {
@@ -55,7 +54,7 @@ tasks.withType<JavaCompile> {
 }
 
 tasks.withType<Test> {
-    @Suppress("SimpleRedundantLet", "UNNECESSARY_SAFE_CALL")
+    @Suppress("SimpleRedundantLet")
     jvmArgs?.let { it.plus("-ea") }
 
     useJUnitPlatform()

diff --git a/buildSrc/src/main/kotlin/faker-pub-conventions.gradle.kts b/buildSrc/src/main/kotlin/faker-pub-conventions.gradle.kts
@@ -7,6 +7,7 @@ import io.github.serpro69.semverkt.gradle.plugin.tasks.TagTask
 plugins {
     `maven-publish`
     signing
+    id("faker-base-conventions")
 }
 
 publishing {
@@ -86,11 +87,3 @@ tasks.withType<Sign>().configureEach {
     onlyIf { !isDev.get() && !isSnapshot.get() }
     onlyIf { isRelease.get() }
 }
-
-//// Run :tag only after we've published artifacts to sonatype
-//tasks.withType<TagTask>().configureEach {
-//    // don't apply when "dryRun"
-//    findProperty("dryRun") ?: run {
-//        dependsOn(rootProject.tasks.getByName("closeSonatypeStagingRepository"))
-//    }
-//}