search

detekt / detekt

Static code analysis for Kotlin
https://detekt.dev
Apache License 2.0
6.07k stars 751 forks source link

Set compiler JDK home to java.home by default #7247

Closed 3flex closed 2 weeks ago

3flex commented 2 weeks ago

This ensures JDK home is always set to a value and aligns to compiler behaviour.

detekt-ci commented 2 weeks ago
Warnings
:warning: It looks like this PR contains functional changes without a corresponding test.

Generated by :no_entry_sign: dangerJS against db5de44d751581258da2e203e5d9a00367b820c9

codecov[bot] commented 2 weeks ago

Codecov Report

Attention: Patch coverage is 50.00000% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 84.71%. Comparing base (0b3edd7) to head (db5de44). Report is 1 commits behind head on main.

Files Patch % Lines
.../io/github/detekt/parser/KotlinEnvironmentUtils.kt 33.33% 1 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #7247 +/- ## ========================================= Coverage 84.71% 84.71% Complexity 3992 3992 ========================================= Files 578 578 Lines 12162 12163 +1 Branches 2495 2494 -1 ========================================= + Hits 10303 10304 +1 - Misses 625 626 +1 + Partials 1234 1233 -1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.