Skip to content

Fix and simplify Maven Central credentials - #3086

Merged
nedtwigg merged 2 commits into
mainfrom
g/20260916/fix-snapshots
Sep 17, 2026
Merged

nedtwigg merged 2 commits into
mainfrom
g/20260916/fix-snapshots

Conversation

@Goooler

@Goooler Goooler commented Sep 16, 2026

Copy link
Copy Markdown
Member

Try to fix https://github.com/diffplug/spotless/actions/runs/35094628080/job/104794839433

* What went wrong:
Credentials required for this build could not be resolved.
> The following Gradle properties are missing for 'mavenCentral' credentials:
    - mavenCentralUsername
    - mavenCentralPassword

@Goooler
Goooler force-pushed the g/20260916/fix-snapshots branch 6 times, most recently from 137b4e4 to c6ae457 Compare September 16, 2026 13:30
Comment thread .github/workflows/ci.yml Outdated
Comment thread .github/workflows/ci.yml
Comment thread .github/workflows/ci.yml Outdated
@Goooler
Goooler force-pushed the g/20260916/fix-snapshots branch 3 times, most recently from d7ef83d to 14bdd87 Compare September 17, 2026 16:28
@Goooler
Goooler force-pushed the g/20260916/fix-snapshots branch from e7b893a to b667bcb Compare September 17, 2026 16:49
@Goooler
Goooler marked this pull request as ready for review September 17, 2026 16:49
Comment on lines -82 to -88
create("spotlessPluginLegacy") {
id = "com.diffplug.gradle.spotless"
implementationClass = "com.diffplug.gradle.spotless.SpotlessPluginRedirect"
displayName = "Spotless formatting plugin (legacy)"
description = project.description
tags = listOf("format")
}

@Goooler Goooler Sep 17, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I recalled there is no need for publishing the legacy plugin, just declare it as the history item in b667bcb. Copied from https://github.com/GradleUp/shadow/blob/35b7a3db0f0d9fa795c8e34557a8dd23e0edf2ac/build.gradle.kts#L93-L108

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

excellent!

@Goooler

Goooler commented Sep 17, 2026

Copy link
Copy Markdown
Member Author

There should be no blockers left for the new release now. Please check and test the release with RCs before the final GA.

@nedtwigg

Copy link
Copy Markdown
Member

Will do! Thanks for this wonderful cleanup - I'm excited for the renovate PRs to "just work"

@nedtwigg
nedtwigg enabled auto-merge September 17, 2026 17:01
@nedtwigg
nedtwigg merged commit 279e210 into main Sep 17, 2026
32 checks passed
@nedtwigg
nedtwigg deleted the g/20260916/fix-snapshots branch September 17, 2026 17:06
dayanruben pushed a commit to dayanruben/spotless that referenced this pull request Sep 18, 2026
diffplug#3086 stopped publishing the marker for the legacy
`com.diffplug.gradle.spotless` id and added a historical capability, both
of which can break a consumer's build. Neither was in the changelog.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
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