Update dependency com.android.tools.build:gradle to v7.4.0

[3flex]

No description provided.

[3flex]

Incompatible because this component declares a component compatible with Java 11 and the consumer needed a component compatible with Java 8

Now we could possibly jump to Java 11 minimum to execute the Gradle plugin, but AGP 8.0 bumps again to Java 17 which will be too aggressive, so we should probably solve for this now and maintain compatibility with Java 8. Hopefully that's feasible because this is a compile only dependency, so we can compile with Java 11 dependencies but produce Java 8 bytecode.

[codecov]

Codecov Report

Merging #5693 (605c4bd) into main (0245b0f) will increase coverage by 84.46%.
The diff coverage is n/a.

❗ Current head 605c4bd differs from pull request most recent head 3b174e7. Consider uploading reports for the commit 3b174e7 to get more accurate results

@@             Coverage Diff             @@
##             main    #5693       +/-   ##
===========================================
+ Coverage        0   84.46%   +84.46%     
- Complexity      0     3734     +3734     
===========================================
  Files           0      543      +543     
  Lines           0    12704    +12704     
  Branches        0     2222     +2222     
===========================================
+ Hits            0    10730    +10730     
- Misses          0      865      +865     
- Partials        0     1109     +1109     

Impacted Files	Coverage Δ
...o/github/detekt/tooling/internal/EmptyContainer.kt	42.85% <0.00%> (ø)
...b/arturbosch/detekt/core/suppressors/Suppressor.kt	71.42% <0.00%> (ø)
...n/kotlin/io/gitlab/arturbosch/detekt/api/Entity.kt	75.00% <0.00%> (ø)
...t/metrics/processors/ProjectComplexityProcessor.kt	100.00% <0.00%> (ø)
...rturbosch/detekt/rules/style/OptionalWhenBraces.kt	94.11% <0.00%> (ø)
...sch/detekt/formatting/wrappers/ModifierOrdering.kt	100.00% <0.00%> (ø)
...osch/detekt/rules/empty/EmptyDefaultConstructor.kt	81.25% <0.00%> (ø)
...turbosch/detekt/rules/style/UseEmptyCounterpart.kt	90.47% <0.00%> (ø)
.../io/github/detekt/tooling/dsl/ConfigSpecBuilder.kt	100.00% <0.00%> (ø)
...tekt/formatting/wrappers/SpacingAroundOperators.kt	100.00% <0.00%> (ø)
... and 533 more

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

[chao2zhang]

We probably need to validate if Android projects can still apply detekt-gradle-plugin if there is internal checks inside AGP. I am referring to https://developer.android.com/studio/releases/gradle-plugin#jvm-11-bytecode

This means that if you compile against AGP, or write custom Lint checks, you need to start targeting JVM 11 bytecode

[BraisGabin]

I don't get how this work. You are not adding the agp to the jdk8 run. Shouldn't those tests fail with class not found or something similar?

[3flex]

This means that if you compile against AGP, or write custom Lint checks, you need to start targeting JVM 11 bytecode

We shouldn't have any problems here, since we're still compiling against AGP 7.0 which doesn't have that limitation, and we're not writing custom lint checks.

[3flex]

Shouldn't those tests fail with class not found or something similar?

Android tests are already skipped if JVM is lower than 11.

detekt/detekt-gradle-plugin/src/functionalTest/kotlin/io/gitlab/arturbosch/detekt/DetektAndroidSpec.kt

Lines 13 to 15 in 0245b0f

	@EnabledForJreRange(min = JRE.JAVA_11, disabledReason = "Android Gradle Plugin 7.0+ requires JDK 11 or newer")
	@EnabledIf("io.gitlab.arturbosch.detekt.DetektAndroidSpecKt#isAndroidSdkInstalled")
	class DetektAndroidSpec {

[BraisGabin]

I really like this addition to compile against min versión but tests against last know version :)

[BraisGabin]

<3

[chao2zhang]

❤️