Dmitry Suzdalev
Dmitry Suzdalev
I agree! It would be easy to implement for primitive types, but it seems that it would require the type resolution enabled for your example with a sealed class? Otherwise...
Hmm. But this rule tries to nudge you to change the first case into letting the parent supply the constant while simplifing the implementation of `Foo`: ```kotlin fun main() {...
These were two very interesting and nice episodes, thanks to you and David!
I had exactly the same feelings as @ewoks after listening to this episode, by the end I didn't like it at all, but all in all I learned something useful...
It's actually interesting to find out about different opinions on a variety of discussed subjects, i.e. it's interesting to hear each opinion backed up by some explanation on why it...
It seems like this has been fixed by #4881 ? At least I found the corresponding test case [here](https://github.com/detekt/detekt/blob/d5165629636eb5d37fd10b10761c9363adb808e8/detekt-rules-style/src/test/kotlin/io/gitlab/arturbosch/detekt/rules/style/UnnecessaryParenthesesSpec.kt#L296).
+1 I have experienced this issue too, caught it with Leak Canary.
This seems to be a duplicate of #1360 (but this report is more detailed). Also I discovered the same error when using dependencies in a multiplatform project. Not sure if...
@cgruber Sorry for me-too-ing, but any idea if this wrapper of yours will be available sometime to use? Maybe in some unfinished form. It's just that I had a pet...
Currently it doesn't work in `jvm`-only KMP modules: ```kotlin kotlin { jvm() sourceSets { named("jvmMain") { dependencies { implementation("com.squareup.logcat:logcat:0.1") } } } } ``` The above configuration results in `import...