Skip to content
New issue

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

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

Already on GitHub? # to your account

Fix Gradle 8.12 deprecation warning #455

Merged
merged 2 commits into from
Dec 18, 2024

Conversation

breskeby
Copy link
Contributor

@breskeby breskeby commented Dec 18, 2024

While testing gradle 8.12 rc 1 & 2 we realised they introduced a few more deprecations and that broke our build.

This fixes the deprecation warnings we saw. We use jitpack to test this in our build here: elastic/elasticsearch#118683

  • We should not reference project from task execution phase

- We should not reference project from task execution phase
breskeby added a commit to breskeby/elasticsearch that referenced this pull request Dec 18, 2024
@breskeby
Copy link
Contributor Author

@rpalcolea do you have the bandwidth to review this one?

@rpalcolea rpalcolea merged commit 9eeb25c into nebula-plugins:main Dec 18, 2024
6 checks passed
@rpalcolea
Copy link
Member

oh this is nice! thanks for doing it. I was planning to go through some deprecations after the holiday season! will release in a bit

@rpalcolea
Copy link
Member

@rpalcolea
Copy link
Member

alright, this is now available @breskeby https://plugins.gradle.org/plugin/com.netflix.nebula.ospackage ! thanks again!

@breskeby
Copy link
Contributor Author

Wow that was quick! highly appreciated!

@breskeby breskeby deleted the fix-812-deprecations branch December 18, 2024 18:27
breskeby added a commit to breskeby/elasticsearch that referenced this pull request Dec 23, 2024
breskeby added a commit to breskeby/elasticsearch that referenced this pull request Dec 23, 2024
breskeby added a commit to breskeby/elasticsearch that referenced this pull request Dec 24, 2024
breskeby added a commit to breskeby/elasticsearch that referenced this pull request Dec 27, 2024
breskeby added a commit to breskeby/elasticsearch that referenced this pull request Dec 30, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants