Skip to content

chore(ci-cd): publish to maven local does not require sign key - #82

Merged
rawo merged 1 commit into
mainfrom
chore/publish-maven-local
Jul 28, 2026
Merged

chore(ci-cd): publish to maven local does not require sign key#82
rawo merged 1 commit into
mainfrom
chore/publish-maven-local

Conversation

@rawo

@rawo rawo commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Problem

Publishing artefacts to maven central required sign key. However publishing to local .m2 repository should not required the sign key.

Solution

signAllPublications() // Only sign when a key is actually configured (CI's release job sets this via // ORG_GRADLE_PROJECT_signingInMemoryKey). Signing unconditionally breakspublishToMavenLocal // for local development/testing, since no GPG key is available outside the release job. if (project.hasProperty("signingInMemoryKey")) { signAllPublications() }

Copilot AI review requested due to automatic review settings July 28, 2026 12:07

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Gradle publishing convention to avoid requiring a signing key when publishing artifacts to the local Maven repository (~/.m2), while preserving signing for Maven Central releases in CI.

Changes:

  • Makes publication signing conditional on the presence of the signingInMemoryKey Gradle property.
  • Adds inline documentation explaining why unconditional signing breaks publishToMavenLocal.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@rawo
rawo merged commit 8fc819d into main Jul 28, 2026
9 checks passed
@rawo
rawo deleted the chore/publish-maven-local branch July 28, 2026 12:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants