Use gradle property to control signing.required (#123)

Adds gradle property signingRequired. This allows developers to use `./gradlew publishToMavenLocal` instead of `./gradlew publishToMavenLocal -x signMavenJavaPublication`. See deephaven/deephaven-core#3846 See gradle/gradle#24456
deephaven · May 22, 2023 · 3605edc · 3605edc
1 parent 19893fd
commit 3605edc
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/.github/workflows/publish.yml b/.github/workflows/publish.yml
@@ -44,3 +44,4 @@ jobs:
           ORG_GRADLE_PROJECT_ossrhPassword: ${{ secrets.SONATYPE_PASSWORD }}
           ORG_GRADLE_PROJECT_signingKey: ${{ secrets.CI_AT_DEEPHAVEN_KEY }}
           ORG_GRADLE_PROJECT_signingPassword: ${{ secrets.CI_AT_DEEPHAVEN_PASSWORD }}
+          ORG_GRADLE_PROJECT_signingRequired: true
diff --git a/buildSrc/src/main/groovy/io.deephaven.csv.java-publishing-conventions.gradle b/buildSrc/src/main/groovy/io.deephaven.csv.java-publishing-conventions.gradle
@@ -80,6 +80,7 @@ publishing {
 }
 
 signing {
+    required = "true" == findProperty('signingRequired')
     sign publishing.publications.mavenJava
     String signingKey = findProperty('signingKey')
     String signingPassword = findProperty('signingPassword')