Skip to content

State ./gradlew spotlessApply fixes all issues in error message#2593

Merged
nedtwigg merged 6 commits intodiffplug:mainfrom
veyndan:rm-path-error-message
Sep 23, 2025
Merged

State ./gradlew spotlessApply fixes all issues in error message#2593
nedtwigg merged 6 commits intodiffplug:mainfrom
veyndan:rm-path-error-message

Conversation

@veyndan
Copy link
Copy Markdown
Contributor

@veyndan veyndan commented Aug 8, 2025

edited by @nedtwigg: prefix issue numbers with a bullet for inline-expansion

@nedtwigg
Copy link
Copy Markdown
Member

Hmm, why :spotlessApply instead of spotlessApply? I don't think :spotlessApply will work, it only applies to the root project.

@veyndan
Copy link
Copy Markdown
Contributor Author

veyndan commented Aug 17, 2025

Ah you're totally right, it should be spotlessApply. I originally had it like that, but updated it to :spotlessApply because it looked like the tests were failing, and wasn't really thinking about the real differences between the two tasks.

I just pushed a revert of this change. The tests will still fail though — happy if anyone wants to try and fix them, otherwise I'm back from vacation in a week and a bit, so happy to fix them up when I'm back.

@nedtwigg nedtwigg enabled auto-merge September 23, 2025 05:57
@nedtwigg nedtwigg merged commit ed0d468 into diffplug:main Sep 23, 2025
19 checks passed
@veyndan veyndan deleted the rm-path-error-message branch September 23, 2025 07:59
@nedtwigg
Copy link
Copy Markdown
Member

Published in plugin-gradle 8.0.0.

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.

Specify that ./gradlew spotlessApply will fix all issues in error message

2 participants