search

detekt / detekt

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

Remove PsiFile.basePath #7237

Closed 3flex closed 1 week ago

codecov[bot] commented 1 week ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 84.78%. Comparing base (65f3164) to head (cb3d8f5).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #7237 +/- ## ========================================= Coverage 84.77% 84.78% - Complexity 3993 3994 +1 ========================================= Files 578 578 Lines 12138 12136 -2 Branches 2484 2483 -1 ========================================= - Hits 10290 10289 -1 + Misses 624 623 -1 Partials 1224 1224 ```

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

BraisGabin commented 1 week ago

Probably we should forbid any usage of absolute() or similar functions called from our core or reporters.

It's not doable right now but it would prevent issues in the future. I'll add that as soon as I finish with option to instantiate the same rule more than once.