Thread
#detekt
    d

    dimsuz

    9 months ago
    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.
    gammax

    gammax

    9 months ago
    You're correct and no, there is no issue to track 😦
    s

    schalkms

    9 months ago
    It's actually supported by detekt. https://github.com/detekt/detekt/pull/3453
    gammax

    gammax

    9 months ago
    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

    dimsuz

    9 months ago
    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

    schalkms

    9 months ago
    Ups sorry.. I overlooked the
    commonMain
    comment.