search

detekt / detekt

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

Move report base path handling to OutputReport #7234

Closed 3flex closed 2 weeks ago

3flex commented 2 weeks ago

Base path is only relevant for report output. This opens the next step of removing FilePath class and using absolute paths everywhere else.

codecov[bot] commented 2 weeks ago

Codecov Report

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

Project coverage is 84.66%. Comparing base (537c4fe) to head (25aa867).

Files Patch % Lines
...otlin/io/github/detekt/report/md/MdOutputReport.kt 75.00% 1 Missing and 1 partial :warning:
...n/io/github/detekt/report/html/HtmlOutputReport.kt 75.00% 0 Missing and 1 partial :warning:
...lin/io/github/detekt/report/xml/XmlOutputReport.kt 80.00% 0 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #7234 +/- ## ============================================ - Coverage 84.68% 84.66% -0.03% - Complexity 3984 3992 +8 ============================================ Files 578 578 Lines 12158 12168 +10 Branches 2499 2499 ============================================ + Hits 10296 10302 +6 - Misses 626 627 +1 - Partials 1236 1239 +3 ```

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