Out of curiosity, why does detekt’s studio plugin set a max version?

Is there a way to set the Kotlin language version that should be used when analyzing things like deprecation? I'm using detekt in an project that uses Gradle Kotlin DSL which has a language version of 1.4 where

String.capitalize(Locale)

isn't deprecated, and the IDE doesn't warn me about deprecation, but detekt does.

With the new 1.22.0-RC1 version I'm getting a lot of "requires type resolution but it was run without it"-like output. How can I avoid that?

Is someone else seeing occasional

java.lang.StackOverflowError (no error message)

with 1.22.0-RC1?

Hi, out of nowhere I'm seeing this error message if I try to run detekt in a mulit-module project's submodule:

> Run failed with 1 invalid config property.
        - Property 'formatting' is misspelled or does not exist.

I have a fairly large existing android project. I want to add detekt. so I added id("io.gitlab.arturbosch.detekt").version("1.22.0-RC1") to my root build.gradle. I ran ./gradlew detekt and it ran successfully, but I would have figured it would have failed as I'm sure some of my project is breaking some detekt rules. Am I missing something? Sorry I'm a detekt noob.

I just noticed that

detektDebug

task doesn't work on my CI, because each module gets a

NO-SOURCE

status:

Task :feature:catalog:ui:detektDebug NO-SOURCE

But this works and detects issues when I run the same

detektDebug

task locally. Any ideas what might cause this? ~UPD: Can this happen because I run detekt tasks on CI before anything gets compiled?~Nope, doesn't work after compile either. UPD2: Looks like I have found the reason. Our CI builds use gradle remote cache, so all lines which have

NO-SOURCE

, have something like this before them:

Task :feature:catalog:ui:compileDebugKotlin FROM-CACHE

And this causes detekt to not run any checks (although there are some failing). Is this a known bug? Can't seem to find it right away on github.

Why do we need the dependency on jar file in

generateDocumentation

task? https://github.com/detekt/detekt/blob/59bedc9eeda131b404dda44330e5cf4645e81fa1/detekt-generator/build.gradle.kts#L46

file("${rootProject.rootDir}/detekt-generator/build/libs/detekt-generator-${Versions.DETEKT}-all.jar"),

detekt participates on the hacktoberfest. We have a lot of

help wanted

issues on our issue tracker so feel free to grab some and get that T-Shirt 😉

Hi everyone, my actual company has some custom plugin that integrate detekt in our Android projects, I successfully run detekt but now I’m trying to add the formatting, the plugins necessary are inside the custom plugin but when i try to run detekt I receive this message: “Provider io.gitlab.arturbosch.detekt.formatting.FormattingProvider could not be instantiated” Any hint to solve the problem?

Hello, I have an interface for

Firebase

events and I want to check whether or not the passed

id

parameter exceeds 40 char limit. What function of a custom detekt rule(e.g.

visitClassBody

) should I use? Sample usages of different calls So how can I access to the

create

function and check its

id

property?

interface FirebaseTrackingEvent : TrackingEvent {
    companion object {
        fun create(firebaseId: String, params: Map<String, Any> = emptyMap()): FirebaseTrackingEvent {
            return FirebaseTrackingEventDataClass(firebaseId, params)
        }
    }

object SampleTrackingEvents {

    val sampleEvent = FirebaseTrackingEvent.create(id = "sample_event_id_which_is_more_than_40_char")

    fun sampleEvent1(tabIndex: Int): TrackingEvent {
       return FirebaseTrackingEvent.create(
           firebaseId = "sample_event_id_which_is_more_than_40_char",
           params = mapOf("index" to tabIndex)
       )
   }
}

Hi folks. I wanted to work on this issue for IDEA plug-in to enable support of multiple baselines for multi-modular projects, but have some trouble understanding how to better implement this. As far as I understand Detekt CLI, which is being invoked from the plug-in, supports only one baseline file. My idea was to use

BaselineProvider

to load all baselines from multiple modules, collect all findings and manually suppressed issues into two single sets and create new baseline from them, encapsulating all issues from individual files into one. When I would have stored this file somewhere locally and use it to pass to Detekt CLI. But now I see that there is an intention to add a new finding into baseline. In current scenario it should modify provided baseline file. But as we supposed to have multiple config files it’s a question, which config file to update. Any idea about this?

Hi. What is the motivation to copy over ktlint rules? We always assumed detekt to be semantics checker and ktlint to be formatting checker. Does detekt want to replace ktlint?

hi, I want to integrate detekt formatting into my Android app but it’s not very clear for me what steps to follow from the documentation… the docs in GitHub just mention a dependency block in gradle and that’s it. Is there any gradle task like “ktLintFormat” that is added that I can use to automatically format my code for example? or how does it works? thank you in advance and sorry for the complete ignorance on this matters, I’m kind of new to all this tools

Hi , Detekt wrapper for ktlint does not check or format build.gradle file like ktlint do. is there a way to enable this ?

Is there parameter for detekt or some setting, to skip check sources or specific sourse set if it haven't changed?

Hi there! Is there a different way to exclude certain folders for KMP projects? Currently with

exclude("**/build/**", "**/generated/**", "**/resources/**")

for my

Detekt

task it doesn't ignore any of these directories. My detekt setup: https://pastebin.com/Lf8xjHdn

Is there any way to get a preview build of detek for evaluating this? https://kotlinlang.slack.com/archives/CKS3XG0LS/p1667248621218719?thread_ts=1667248621.218719&cid=CKS3XG0LS

Running into issues when running

detektMain

on a project that has Gradle precompiled script plugins:

Front-end Internal error: Failed to analyze declaration Co_myapp_kotlin_multiplatform_gradle
  File being compiled: (1,71) in //home/eli/workspace/myapp/src/main/kotlin/co.myapp-kotlin-multiplatform.gradle.kts
  The root cause org.jetbrains.kotlin.resolve.lazy.NoDescriptorForDeclarationException was thrown at: org.jetbrains.kotlin.resolve.lazy.BasicAbsentDescriptorHandler.diagnoseDescriptorNotFound(AbsentDescriptorHandler.kt:18)

Is this a known issue, or should I file something in GitHub?

Hi, I have already asked a question here today: https://github.com/detekt/detekt/discussions/5499 But am trying to solve the problem differently now. For this I need to know the following: Can someone tell me how to enable type resolution in this detektBaselineMultiModule: https://detekt.dev/docs/introduction/baseline/#kotlin-dsl ? I tried it with

jvmTarget = "1.8"

The problem is that now I want to use detektMain and detektTest, in these type resolution is enabled, but in detektBaselineMultiModule it is not.

hello , i want to know how to use detekt with formatting plugin in a multi module project, in a way to include/exclude sub projects , should detekt configuration and dependencies be present in each sub project build file ? or just in the root build file ? i tried but i keep getting this error :

Execution failed for task ':app:detekt'.

Run failed with 1 invalid config property.

- Property 'formatting' is misspelled or does not exist.

If you specify multiple files to the

config

Gradle property, will it merge the configs together? Or will one override the other? I'm trying to find a way to have a common config with shared options and then separate configs for different modules with more specific settings and options, to avoid repeating myself everywhere.

Just so I'm not mistaken: There is currently no way to do a reference search in a Detekt rule or is there?

Hi! I'm thinking about writing a rule which would nudge the programmer to use exhaustive

when

on enums, e.g.

enum Variant { One, Two, Three }

// non-compliant
if (variant == Variant.One) 
  doStuff() else doOtherStuff()

// compliant
when (variant) {
  One -> doStuff()
  Two, Three -> doOtherStuff()
}

with rationale being that in the latter case if

Four

would be added it's better to have the compiler tell you that you should think about how you would handle it in every place you depend on this enum. But first I wanted to ask: is there something like this in Detekt's rule set already?

Would it be possible to stop making breaking config changes in minor version bumps? I have a lot of projects that use detekt, and every time there's a new version I have to manually go update the config (which despite being simple can get tedious).

I'm thinking to implement a rule that it have sense to me in my experience but I'm not sure if it's backed up by any book, good practices, general consensous... I would like a rule that forbids to extend a

Throwable

with a

data class

. Example

data class Foo(val status: Int) : IllegalStateException()

. The reason is that any

Throwable

stores information about were it was instantiated. For that reason the

equals

that

data class

implements is missleading. Two equal instances of

Foo

have different behaviour. My question: do you know any source talking about this? Does a rule like this have sense to you?

Is detekt able to analyse non-kotlin files, for example the meta-inf files like service providers? Use-case: if you implement a service provider you need to have a non-arg constructor

Hello, World! A PSI / type resolution question. I'm implementing

visitCallExpression(expression: KtCallExpression)

, trying to find calls to a specific extension function. When calling

expression.getResolvedCall(bindingContext)

on a regular function, I'm correctly getting the corresponding descriptor. However, when the call is an extension function, this returns

null

. Any idea if there's a known limitation or another way to do this?

With the new detekt 1.22 I cannot seem to be able to

@Suppress("EnumEntryNameCase")

, is anyone else having that problem?

Hey everyone, Has anyone ever tried leveraging detekt for providing refactor scripts? Detekt easily gives me access to

KtElement

hierarchy. Making it quite easy to inspect and find certain usages, I was wondering if there was something like "auto-fix" like KtLint auto-formatting I can leverage as well. Otherwise, just finding certain usages and modifying the files is also an option 🤔