fred
12/07/2022, 9:17 AMdetektMain
to sweep non-multiplatform stuff with type resolution
◦ detektMetadataMain
to sweep commonMain
on all modules (found this one here)
◦ detektAndroidDebug
and detektAndroidRelease
to sweep Android code in multiplatform modules
◦ …and how do I sweep iOS code in multiplatform modules? I’ve tried detektMetadataIosMain
, detektIosX64Main
, detektIosArm64Main
and detektIosSimulatorArm64Main
but they all seem to fail to find issues on code inside iosMain
— should I file an issue or am I missing something? I have a minimum reproducible example here: https://github.com/tfcporciuncula/detekt-kmm-issue (this commit summarizes everything)
• any other option?Mustafa Ozhan
12/07/2022, 11:50 AMgammax
12/07/2022, 12:48 PM• run multiple tasks:That’s the current status. We generate tasks for each module and for each Multiplatform sourceset.
should I file an issue or am I missing something?Potentially yes + a reproducer would be gold. We added KMP support ~1/2 years ago and never received any feedback if it actually worked or was valuable for users, so happy to iterate on it.
Mustafa Ozhan
12/07/2022, 1:38 PMwas valuable for usersDefinitely is, I think the reason of lacking feedback due to not having successful KMP adaptation by developers Maybe we should create a new single task that depend on the relevant tasks in activated platforms ? I can file an issue for that.
fred
12/07/2022, 2:28 PMtasks.withType<io.gitlab.arturbosch.detekt.Detekt> detekt@{
setSource(files(project.projectDir))
exclude("**/*.kts")
exclude("**/build/**")
}
with this I can just run detekt
and make sure it covers the entire project (with no type resolution)eygraber
12/07/2022, 3:33 PMeygraber
12/07/2022, 3:33 PM