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

Make plugin work under Gradle 7 #46

Merged
merged 1 commit into from
Apr 13, 2021
Merged

Make plugin work under Gradle 7 #46

merged 1 commit into from
Apr 13, 2021

Conversation

sgrimm
Copy link
Contributor

@sgrimm sgrimm commented Apr 10, 2021

Gradle 7 requires that task properties be annotated to indicate how they should be
treated when calculating what artifacts need to be rebuilt. One of the methods in
the gradle-processes plugin was missing an annotation, which caused the plugin to
bomb out under Gradle 7.

Fix it by subclassing the task in question and annotating the method.

Gradle 7 requires that task properties be annotated to indicate how they should be
treated when calculating what artifacts need to be rebuilt. One of the methods in
the gradle-processes plugin was missing an annotation, which caused the plugin to
bomb out under Gradle 7.

Fix it by subclassing the task in question and annotating the method.
@sgrimm
Copy link
Contributor Author

sgrimm commented Apr 10, 2021

There's an open issue #36 to replace the gradle-processes plugin entirely, but while that change is being considered, this one can serve as a stopgap to at least keep the plugin usable now that Gradle 7 is officially released.

@nhajratw
Copy link

thanks for making this PR. I had created issue #47 before i saw it. Hope it gets merged soon!

@bnasslahsen bnasslahsen merged commit 49ea922 into springdoc:master Apr 13, 2021
@sgrimm sgrimm deleted the gradle7 branch April 13, 2021 17:52
@bnasslahsen
Copy link
Contributor

@sgrimm,

Thank you for your initiative and contribution.

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.

3 participants