Skip to content

Accept Gradle properties for Sonatype + signing credentials#4

Merged
atanasiuk-hubspot merged 3 commits into
mainfrom
atanasiuk/oss-release-credentials
Apr 30, 2026
Merged

Accept Gradle properties for Sonatype + signing credentials#4
atanasiuk-hubspot merged 3 commits into
mainfrom
atanasiuk/oss-release-credentials

Conversation

@atanasiuk-hubspot
Copy link
Copy Markdown
Contributor

Why: the HubSpotEngineering/oss-release Gradle release flow exports credentials as ORG_GRADLE_PROJECT_mavenCentralUsername/Password and ORG_GRADLE_PROJECT_signingInMemoryKey{,Id,Password}. The current publish/signing config only reads SONATYPE_* / GPG_SIGNING_* env vars, so the shared release job can't authenticate to Sonatype or sign jars.

Reading via findProperty(...) with the existing env vars as a fallback keeps the standalone publish-sonatype.yml workflow working unchanged while letting oss-release drive a release without a wrapper.

Why: the HubSpotEngineering/oss-release Gradle release flow exports
credentials as ORG_GRADLE_PROJECT_mavenCentralUsername/Password and
ORG_GRADLE_PROJECT_signingInMemoryKey{,Id,Password}. The current
publish/signing config only reads SONATYPE_* / GPG_SIGNING_* env vars,
so the shared release job can't authenticate to Sonatype or sign jars.

Reading via findProperty(...) with the existing env vars as a fallback
keeps the standalone publish-sonatype.yml workflow working unchanged
while letting oss-release drive a release without a wrapper.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@atanasiuk-hubspot atanasiuk-hubspot marked this pull request as ready for review April 30, 2026 11:53
@atanasiuk-hubspot atanasiuk-hubspot merged commit 49495db into main Apr 30, 2026
6 of 7 checks passed
@atanasiuk-hubspot atanasiuk-hubspot deleted the atanasiuk/oss-release-credentials branch April 30, 2026 12:02
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