Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BREAKING CHANGE: Gradle Java Toolchains support for JavaExec and WorkerAPI #526

Closed
wants to merge 3 commits into from

Conversation

rpalcolea
Copy link
Contributor

@rpalcolea rpalcolea commented Jul 13, 2021

With the introduction of Toolchains for JVM Projects, there might be some folks that would like to leverage such feature when executing spotbugs either via worker api / javaexec / hybrid.

This pull request adds support for it. For now, this requires explicit opt-in via

spotbugs {
          useJavaToolchains = true
}

If necessary, this could be default in the future but it seems like a safer approach to roll out this feature.

It will be interesting to keep an eye on gradle/gradle#16791

This is a breaking change as Toolchain support was introduced in Gradle 6.7 and became mature in 7.x

I suspect this should be rolled out as maybe 7.+ only? thoughts?

@rpalcolea rpalcolea force-pushed the toolchains-support branch from dd44e0f to 89e8626 Compare July 13, 2021 17:38
@rpalcolea rpalcolea force-pushed the toolchains-support branch from 89e8626 to 32f780b Compare July 13, 2021 17:42
@rpalcolea rpalcolea changed the title feat: Gradle Java Toolchains support for JavaExec and WorkerAPI Gradle Java Toolchains support for JavaExec and WorkerAPI Jul 13, 2021
@jscancella
Copy link
Contributor

LGTM except for the latest commit message not being a semantic versioning tag

@rpalcolea rpalcolea changed the title Gradle Java Toolchains support for JavaExec and WorkerAPI BREAKING CHANGE: Gradle Java Toolchains support for JavaExec and WorkerAPI Jul 13, 2021
@rpalcolea rpalcolea force-pushed the toolchains-support branch from 32f780b to 2112597 Compare July 13, 2021 17:50
@rpalcolea rpalcolea force-pushed the toolchains-support branch from 2112597 to 4173f41 Compare July 13, 2021 17:57
@rpalcolea
Copy link
Contributor Author

Updated the commits and intentio nbehind the change 👍

jscancella
jscancella previously approved these changes Jul 14, 2021
@KengoTODA KengoTODA mentioned this pull request Sep 16, 2021
KengoTODA added a commit that referenced this pull request Sep 17, 2021
close #526

BREAKING CHANGE: Gradle v5 and v6 are now not supported
@github-actions
Copy link

🎉 This issue has been resolved in version 5.0.0-beta.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@KengoTODA
Copy link
Member

I've merged these changes to the beta branch manually. Thanks for your contribution!

@KengoTODA KengoTODA closed this Sep 17, 2021
@github-actions
Copy link

github-actions bot commented Dec 6, 2021

🎉 This issue has been resolved in version 5.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants