-
Notifications
You must be signed in to change notification settings - Fork 439
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
Gradle plugin - improve compatibility with Configuration Cache #1825
Gradle plugin - improve compatibility with Configuration Cache #1825
Conversation
@aSemy Please sign the Contributor License Agreement! Click here to manually synchronize the status of this Pull Request. See the FAQ for frequently asked questions. |
@aSemy Thank you for signing the Contributor License Agreement! |
hey @shanman190 , WDYT? |
@aSemy can you please resolve the conflicts? |
@marcingrzejszczak, the implementation looks good to me and is compatible with the Gradle 6.8 baseline for 3.1. Only thing that stuck out to me was the code style not following the Spring defaults, but that's minor. |
@aSemy can we merge this? |
@marcingrzejszczak definitely! |
Congrats @aSemy on your first contribution :) |
Avoid using Project during task execution
buildRootPath()
, fix some Javadoc)The tests pass, but I haven't verified that the plugin is now compatible with the Gradle Configuration Cache, so I'm marking this as 'draft'.
(hopefully) fixes #1824