Róbert Papp

I also think it's good to keep this, because it would easily catch `data class FooMapper`, which is a true positive finding. Implement "allow operators" flag works I think for...

Based on the upvotes I think you're good to go. Maybe summarise your understanding of the requirements as you see it from above before you start to prevent misunderstandings.

I'm not sure about the list of operators, that complicates things (e.g. have to list all operators by default and maintain that list), but also makes it really flexible. This...

From DX perspective having an empty list by default is horrid. Imagine that every single user would have to configure Detekt in a way that list all operators functions. Each...

Agree with Brais, in the Kotlin naming ["operators" are the function with `operator` keyword](https://kotlinlang.org/docs/operator-overloading.html). Anything else is just "operator-like" function, or "infix functions". There's no clear definition of what constitutes...

I think that's two different questions: * with delegation/inheritance I also think it is a different issue (can `data class` inherit like this?), if you can give a false negative...

Interesting, I think that's just the question of how we define "allowOperators". It looks like "allowOperators: false" is not equivalent to "denyOperators: true", and that's probably ok, just make sure...

Agreed, but here's a tip: version-control `.idea/detekt.xml` There isn't anything project specific, so you can copy between projects. ```xml ```

@JPUF those two problems look like #631 -> fixed by #640 @wezley98's might be also fixed by https://github.com/cashapp/paparazzi/pull/640, reported also in #624 -> which is tracked in #604 -> fixed...

Based on [this answer](https://stackoverflow.com/a/56043252/253468) there's workaround yet for Java 18+. However, following the [future of the commit mentioned in #205's comment](https://github.com/powermock/powermock/commits/release/2.x/powermock-reflect/src/main/java/org/powermock/reflect/internal/WhiteboxImpl.java), they migrated to using `Unsafe` in https://github.com/powermock/powermock/pull/1026 which may...