search

detekt / detekt

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

Don't duplicate constants #7242

Closed BraisGabin closed 2 weeks ago

BraisGabin commented 2 weeks ago

We had DETEKT_OUTPUT_REPORT_BASE_PATH_KEY on :detekt-api and detekt-core. core can use the value in api to avoid disalignments.

7241 will obsolete this PR but meanwhile it will help to find the usages too.

detekt-ci commented 2 weeks ago
Warnings
:warning: It looks like this PR contains functional changes without a corresponding test.
:warning: This PR is approved with no milestone set. If merged, it won't appear in the detekt release notes.

Generated by :no_entry_sign: dangerJS against ef767dce0fd01cdf074cc1ee153a8f24bb00aa72

codecov[bot] commented 2 weeks ago

Codecov Report

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

Project coverage is 84.70%. Comparing base (0cab28d) to head (ef767dc).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #7242 +/- ## ========================================= Coverage 84.70% 84.70% Complexity 3992 3992 ========================================= Files 578 578 Lines 12164 12164 Branches 2495 2495 ========================================= Hits 10303 10303 Misses 627 627 Partials 1234 1234 ```

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