build: only signing on release (#13)

Signed-off-by: tison <[email protected]>
tisonkun · Oct 14, 2024 · b314e29 · b314e29
1 parent 0744c60
commit b314e29
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 7 deletions.
diff --git a/buildSrc/src/main/kotlin/java-publish.gradle.kts b/buildSrc/src/main/kotlin/java-publish.gradle.kts
@@ -67,8 +67,12 @@ publishing {
     }
 }
 
-if (!System.getenv("GITHUB_WORKFLOW").equals("CI", true)) {
-    signing {
-        sign(publishing.publications["mavenJava"])
+signing {
+    setRequired {
+        val releaseVersion = !version.toString().endsWith("-SNAPSHOT")
+        // signing is required if this is a release version and the artifacts are to be published
+        // do not use hasTask() as this require realization of the tasks that maybe are not necessary
+        releaseVersion && gradle.taskGraph.allTasks.any { it is PublishToMavenRepository }
     }
+    sign(publishing.publications["mavenJava"])
 }
diff --git a/gradle.properties b/gradle.properties
@@ -13,4 +13,4 @@
 # limitations under the License.
 
 group=com.tisonkun.os
-version=0.4.0
+version=0.5.0
diff --git a/plugin-gradle/build.gradle.kts b/plugin-gradle/build.gradle.kts
@@ -77,8 +77,12 @@ publishing {
     }
 }
 
-if (!System.getenv("GITHUB_WORKFLOW").equals("CI", true)) {
-    signing {
-        sign(publishing.publications)
+signing {
+    setRequired {
+        val releaseVersion = !version.toString().endsWith("-SNAPSHOT")
+        // signing is required if this is a release version and the artifacts are to be published
+        // do not use hasTask() as this require realization of the tasks that maybe are not necessary
+        releaseVersion && gradle.taskGraph.allTasks.any { it is PublishToMavenRepository }
     }
+    sign(publishing.publications)
 }