search

detekt / detekt

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

Remove FilePath #7235

Closed 3flex closed 1 week ago

codecov[bot] commented 2 weeks ago

Codecov Report

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

Project coverage is 84.77%. Comparing base (6c1de58) to head (ea7cc78).

Files Patch % Lines
...n/io/github/detekt/report/html/HtmlOutputReport.kt 50.00% 0 Missing and 1 partial :warning:
...otlin/io/github/detekt/report/md/MdOutputReport.kt 50.00% 0 Missing and 1 partial :warning:
...lin/io/github/detekt/report/xml/XmlOutputReport.kt 0.00% 0 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #7235 +/- ## ============================================ + Coverage 84.71% 84.77% +0.05% Complexity 3993 3993 ============================================ Files 578 578 Lines 12163 12138 -25 Branches 2494 2484 -10 ============================================ - Hits 10304 10290 -14 + Misses 626 624 -2 + Partials 1233 1224 -9 ```

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

BraisGabin commented 1 week ago

LGTM. It's strange the usages of user.dir on tests but that's clearly not important and you have a lot of PRs waiting for this.

3flex commented 1 week ago

It's strange the usages of user.dir on tests

They weren't really needed, I've stripped them out. Thanks for your time on this!