Ensure detekt-tooling public API is stable#4112
Conversation
Codecov Report
@@ Coverage Diff @@
## main #4112 +/- ##
=========================================
Coverage 83.51% 83.51%
Complexity 3186 3186
=========================================
Files 461 461
Lines 9095 9095
Branches 1768 1768
=========================================
Hits 7596 7596
Misses 570 570
Partials 929 929 Continue to review full report at Codecov.
|
|
A bit off topic but, should we add detekt-test and detekt-test-utils? For example I use both of those in some custom rules: https://github.com/BraisGabin/detekt-junit-rules/blob/main/build.gradle.kts I'm not sure because they are used to run the tests. With api compatibility should be enough, right? |
I think it's worth having the discussion about which APIs need to be strictly maintained, and which don't. That could be a separate issue/discussion. |
It seems
detekt-toolingshould be considered public API. This ensures the API is not accidentally changed.Modules now checked with the binary compatibility validator: