I think I found a bug (or a missing feature, depending on how you want to call it 😉) - how is one able to exclude only certain rules for tests but not a whole rule-set? When I define the following:

exclude-rule-sets:
exclude-rules:
- 'StringLiteralDuplication'

Then the default applies for

exclude-rule-sets

which is uggly because I did not want the default (same applies when I remove all exclude-rules and only add exclude-rule-sets) The workaround I use is:

exclude-rule-sets:
- 'ignoreDefaultBehaviour'

Is there another way to achieve this?

@Eugen Martynov There is

sonar.kotlin.detekt.reportPaths

which should point to

detekt-checkstyle.xml

Also @heyitsmohit gave a talk on static code analysis and detekt on droidcon NYC, if you haven't seen it -> The talk is quite entertaining and it motivates me to work further on detekt, thanks! 🙂

Is it possible to not fail build if baseline file is missing?

Nope, https://github.com/arturbosch/detekt/pull/1221 is still blocked

hi, is there any rule in detekt that disables usages of

!!

?

Any thoughts on whether or not this would be possible without resolving types? (Summary: If function has an argument of type X, make sure any child call expressions for a function that has parameter of type X provide a value. ) https://github.com/arturbosch/detekt/issues/1300 For example, can I force a visitKtClass before hand to get the necessary metadata for a subsequent rule? ( Something like this processor: https://github.com/arturbosch/detekt/blob/master/detekt-sample-extensions/src/main/kotlin/io/gitlab/arturbosch/detekt/sample/extensions/processors/QualifiedNameProcessor.kt)

i'm seeing some behavior with

UnnecessaryApply

that i'm not sure about.

val view = FrameLayout(context).apply {
    background = ColorDrawable(ContextCompat.getColor(context, R.color.white))
}

This is saying it's unnecessary. But I'm not sure how I'd both set the background and return the

FrameLayout

without an

apply

.

Hey everyone, I wrote this short medium to explain how to integrate detekt in case someone can give it a read and approve I'm not doing anything too wrong 😄 it is quite short. https://proandroiddev.com/detecting-kotlin-code-smells-with-detekt-e79c52a35faf

hey guys, we need to rename

detekt-intellij-plugin

to something else. No plugin and no intellij in the name ... 🙂 How about: detekt4idea? Maybe I need to ask if idea is allowed than 😄

How come we need to rename it?

is the approval of new version of IDEA plugin pending because it needs to be renamed? Would be nice to be able to test soon..

its now just called detekt and is up ^^

I fixed that! All that needed is to rename

NoInternalImportRule

in the yml to

NoInternalImport

. @Artur Bosch, @mauin Can we report warnings about unknown rules in the yml configs? I spent 2 days finding this bug

did RC10 change something related to test-pattern / patterns? i have stuff showing up now that was previously excluded

anybody else think the defaults for ObjectPropertyNaming rule should match the Kotlin style guide?

did anything change with

EndOfSentenceFormat

?

We have a tracking issue to validate all inputs in the config and all Regexes to make these errors easier to spot.

Hey 👋 can anyone help me? https://github.com/arturbosch/detekt/issues/1395

Failed to apply plugin [id 'com.xenoterracide.gradle.bundle.kotlin-lib-defaults']

> Extension of type 'JavaPluginConvention' does not exist. Currently registered extension types: [ExtraPropertiesExtension, XenoExtension, IdeaModel, DefaultArtifactPublicationSet, ReportingExtension, SourceSetContainer, JavaPluginExtension, KotlinJvmProjectExtension] any way to get my plugin to wait on that extension?

Hi Ivan! That's actually a nice idea for a detekt rule! Without the source code it's very hard to help you. Can you post a link to your source code please?

At the moment either you have it enabled, in which case it automatically runs on all KT files, or you have it disabled, and it does nothing.

buildscript {
    repositories {
        jcenter()
    }
}

plugins {
    id("io.gitlab.arturbosch.detekt").version("[version]")
}

when will things work without the buildscript? (jcenter()) and/or does anyone know how to add jcenter like this in a plugin via java

but there's no RC12 which is mentioned here https://plugins.gradle.org/plugin/io.gitlab.arturbosch.detekt

also version syntax varies, first it was .RC8 and now -RC10 and -RC12?

Anyone knows any Danger plugin https://danger.systems/js/ that works with detekt (especially for multi-module project)?

Anyone knows whether there’s an IntelliJ plugin for detekt which can run check on a specific Kotlin file from the IDE? For example, right click on the file > run detekt. Then we got a report which is only for that file.

I think parameter

formatting.MaximumLineLength

is redundant: https://github.com/arturbosch/detekt/blob/master/detekt-cli/src/main/resources/default-detekt-config.yml#L222 He can use the parameter `style.MaxLineLength`: https://github.com/arturbosch/detekt/blob/master/detekt-cli/src/main/resources/default-detekt-config.yml#L445? Or at least have the same name ...

just a heads up, keep up the great work! it’s especially important now that ktlint is on hyatus

Please state your opinion in the issue to have one centralized place