Jonathan Bluett-Duncan
Jonathan Bluett-Duncan
Woo, fantastic, great to hear! 🎉
Let me know if or when this issue has been assigned to me, and I'd be more than happy to look into a fix. 😄
Oh, I've just seen that the [contribution guidelines](https://github.com/openrewrite/.github/blob/main/CONTRIBUTING.md#contributing-fixes) say I can open a draft PR anytime I want, so I'll proceed when I have something to show.
@shanman190 Thanks for your insight. I'll wait until I've received the go ahead to bump the min Gradle version to a more suitable version, then.
Actually I've done some research and found out the minimum version needed to run the configuration cache is Gradle 6.6. I'll assume this is fine and proceed with my draft...
Interestingly, on bumping the internal min Gradle version to 6.6, I'm now seeing https://stackoverflow.com/questions/76897459/how-can-i-make-gradle-pick-the-right-guava-variant, which I see @timtebeek responded to, but I don't think an issue has been raised yet....
The easiest solution is to require Gradle 6.8.3+, because we already import the right "attribute" in `ResolveRewriteDependenciesTask.java` but the tests fail on Gradle 6.6 for some reason, and Gradle 6.8.3...
If we do proceed, `DefaultProjectParser` is going to be the biggest challenge, as it relies on `Project` for many things, including getting a `JavaPluginConvention` and getting the root project and...
@timtebeek Thank you very much for the latest news! I'll update #228 to use Gradle 4.10, then.
Blocked until I've received the go ahead to bump the min Gradle version to one that better supports the configuration cache, as per https://github.com/openrewrite/rewrite-gradle-plugin/issues/227#issuecomment-1705710548.