Gradle 8.2 deprecated conventions and added nagging

[TWiStErRob]

From this plugin every user will get this:

> Configure project :
The Project.getConvention() method has been deprecated. This is scheduled to be removed in Gradle 9.0. Consult the upgrading guide for further information: https://docs.gradle.org/8.2-milestone-1/userguide/upgrading_version_8.html#deprecated_access_to_conventions
        at org.gradle.api.internal.project.DefaultProject.getConvention(DefaultProject.java:593)
        at io.github.gradlenexus.publishplugin.NexusPublishPlugin.addMavenRepositories(NexusPublishPlugin.kt:273)
        at io.github.gradlenexus.publishplugin.NexusPublishPlugin.access$addMavenRepositories(NexusPublishPlugin.kt:40)
        at io.github.gradlenexus.publishplugin.NexusPublishPlugin$configurePublishingForAllProjects$1$1$1.execute(NexusPublishPlugin.kt:157)
        at io.github.gradlenexus.publishplugin.NexusPublishPlugin$configurePublishingForAllProjects$1$1$1.execute(NexusPublishPlugin.kt:40)

The Project.getConvention() method has been deprecated. This is scheduled to be removed in Gradle 9.0. Consult the upgrading guide for further information: https://docs.gradle.org/8.2/userguide/upgrading_version_8.html#deprecated_access_to_conventions
	at org.gradle.api.internal.project.DefaultProject.getConvention(DefaultProject.java:593)
	at io.github.gradlenexus.publishplugin.NexusPublishPlugin.configureTaskDependencies(NexusPublishPlugin.kt:280)
	at io.github.gradlenexus.publishplugin.NexusPublishPlugin.access$configureTaskDependencies(NexusPublishPlugin.kt:40)
	at io.github.gradlenexus.publishplugin.NexusPublishPlugin$configurePublishingForAllProjects$1$1$1.execute(NexusPublishPlugin.kt:173)
	at io.github.gradlenexus.publishplugin.NexusPublishPlugin$configurePublishingForAllProjects$1$1$1.execute(NexusPublishPlugin.kt:40)

There isn't much to do to fix this, only we need to compile the project with Gradle 8.2 as suggested here:
gradle/gradle#25206

However, using Gradle 8.2 is not trivial, it requires these steps (and shows how useful it is to merge Renovate PRs soon-ish 😉):

• Merge chore(deps): update plugin com.gradle.enterprise to v3.13.3 #194 (trivial)
• Merge Shadow 8.1.1 #210 (quick review)
  which supersedes chore(deps): update plugin com.github.johnrengelman.shadow to v8 - autoclosed #197
• Merge chore(deps): update plugin org.ajoberstar.stutter to v0.7.2 #225
  which supersedes chore(deps): update plugin org.ajoberstar.stutter to v0.7.2 #195
• Renovate will rebrand this PR to 8.2 as soon as it's released:
  chore(deps): update dependency gradle to v8.1.1 - autoclosed #214 (trivial)
• Release 1.3.1 (if there are no other breaking changes) Ivy changes in Support publishing using Ivy #222 are breaking.

@szpak Timeline to do this is a few weeks until Gradle 8.2 becomes stable.

[szpak]

Thanks @TWiStErRob . I will work on that "soon", after #222 is merged. Btw, what other (potentially non-backward incompatible) changes would you see before 2.0.0?

[maczikasz]

FYI the PR #222 will actually fix that nagging for 8.2

[TWiStErRob]

Cool, good to hear @maczikasz. Note: there were 2 or 3 the calls. That means this is not urgent, but still would be nice to unblock contributions behind an epic PR :)

[TWiStErRob]

@szpak none of what I listed is backwards incompatible IMO. I don't think there's too much need to be afraid of backwards compatibility. Just provide a good migration guide, and then it takes 3 minutes to do a breaking upgrade. 😉 e.g. Stutter's 0.7 is a good example, it's totally changed, yet nothing really changed so it was easy.

[TWiStErRob]

Closing this as fixed by #222 (see diff), because that PR changed the .the<> calls to extension access. (This was the second option recommended by gradle/gradle#25206)

@szpak add milestone?

[TWiStErRob]

@szpak Can you please release this? Gradle 8.2 is released stable. e.g. Mockito is affected.

[szpak]

Can you give 2.0.0-rc-1 as try? It was released a few weeks ago to test integration with Ivy (and I could push a tag with a release to GitHub...)

[TWiStErRob]

Oh, 🤦‍♂️ I forgot that was released. Please do mark it as pre-release so it shows in Releases view on GitHub.

[szpak]

Done. In fact the tag was there, but I needed to "generate" the changelog and create a pre-release.