Update dependency Gradle to v8.1

[3flex]

https://docs.gradle.org/8.1/release-notes.html

[codecov]

Codecov Report

Merging #5996 (dfb8b9f) into main (ff2f619) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main    #5996   +/-   ##
=========================================
  Coverage     84.66%   84.66%           
  Complexity     3836     3836           
=========================================
  Files           549      549           
  Lines         13065    13065           
  Branches       2305     2305           
=========================================
  Hits          11061    11061           
  Misses          868      868           
  Partials       1136     1136           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[cortinico]

nice :)

[cortinico]

Do we want this inside 1.23? Feels like yes

[3flex]

I think so. It doesn't directly impact build artifacts; the configuration cache test change is just to align with Gradle's more correct behaviour in 8.1.

The build failure on Java 8 is an issue, I'll investigate. It might actually be a regression in Gradle.

[github-actions]

	Warnings
⚠️	This PR is approved with no milestone set. If merged, it won't appear in the detekt release notes.

Generated by 🚫 dangerJS against dfb8b9f

[3flex]

It's not a regression, but an issue that we didn't see previously because of a confluence of reasons that will take me too long to type up.

I've raised gradle/gradle#24765 but in the meantime I've marked the GenerateMavenPom task incompatible with configuration cache, but only on Java 8. That will work around the issue and won't impact most contributors anyway since most will be running Gradle with more modern Java versions.