Detekt swears in the Multiplatform project At the same time kotlinlang #detekt

Detekt swears in the Multiplatform project. At the same time, there are no such errors in IDEA. What to do about it?Detekt swears in the Multiplatform project. At the same time, there are no such errors in IDEA. What to do about it? https://github.com/itbasis/kochange/runs/4278481352?check_suite_focus=true#step:4:42

there was: https://youtrack.jetbrains.com/issue/KTIJ-18408

gammax

11/21/2021, 1:36 PM

What do you mean with Swears?

BorzdeG

11/21/2021, 1:36 PM

example:

error: unresolved reference: Currency

The problem seems to be that some of the classes are in commonMain when checking for jvmMain.

seen https://github.com/detekt/detekt/issues/3664

Javier

11/21/2021, 2:18 PM

I haven't tried to run detekt tasks for specific source sets but I am using detekt in kmp projects with no issues

but I just run gradlew detekt

BorzdeG

11/21/2021, 2:23 PM

detekt

in KMP does not check the source code - you can see this by adding an empty class, for example, to commonMain. You will not receive an EmptyClassBlock error

Javier

11/21/2021, 2:28 PM

it does for me

https://github.com/JavierSegoviaCordoba/run-blocking-kmp/pull/9/files

I even have it configured with GitHub code scanning

but I don't run detektJvmMain, I just run detekt

BorzdeG

11/21/2021, 2:39 PM

because you have prescribed this for all Detekt tasks:

Copy code

setSource(projectDir)
include("**/*.kt")
...

also, apparently you don’t have `expect`/`actual` in your project ...

Javier

11/21/2021, 3:19 PM

I am using it in more projects

with expect and actual too

don't remember the default config I have tho, I did it long time ago hahah

https://github.com/JavierSegoviaCordoba/mokoki/tree/main

there are actual expect there

gammax

11/21/2021, 7:42 PM

Just a small heads up: on KMP projects if you use a plain

detetk

you’re not using type resolution. You should use one of the other

detektJvmMain

and similar tasks to enable type resolution.

👍 1

leandro

11/09/2022, 2:54 PM

on KMP projects if you use a plain detetk you’re not using type resolution.

Is this still true? I wonder what’s the best way to run detekt on KMP modules, and more specifically, across the whole project

19 Views