Judging from source code, detekt currently doesn't...
# detekt
d
Judging from source code, detekt currently doesn't support type resolution in KMM commonMain modules. Is there an issue I could track for this feature? Would be awesome to have.
g
You're correct and no, there is no issue to track :(
s
It's actually supported by detekt. https://github.com/detekt/detekt/pull/3453
g
I’m not entirely sure we support Type Resolution for
commonMain
. I’m sure we do for Android & JVM source sets. IIRC we don’t for common, native & JS srcset but I might be wrong.
d
Yeah, what I've found was this line, which disables type resoulution for common (with a helpful why-comment). A pity though, because I'm integrating detekt in a new kmm project which promises large amounts of code in a
commonMain
which I'd like to check in this nice mode. I can understand that it's not trivial to implement though...
Hmm, but on the other hand I've just found this comment, which says that
commonMain
will be checked by
detektMetadataMain
🙂 Will try it tomorrow.
s
Ups sorry.. I overlooked the
commonMain
comment.