Hi All, I could see a difference between gradle-detekt-plugin (1.9.1) and detekt-sonar-plugin(2.0.0) reported issues... with same config yml file.. also anyone please tell me when can we expect detekt-sonar plugin will be updated based on 1.9.1?

is it a good practice to use both?

I can’t seem to get autocorrect to work. This is what my detekt.gradle file looks like. Any ideas?

apply plugin: 'io.gitlab.arturbosch.detekt'

detekt {
    parallel = true
    config = files("$rootDir/codeQuality/detekt.yml")
    autoCorrect = true
    reports {
        xml {
            enabled = false
        }
        txt {
            enabled = false
        }
    }
}

Hi all - I can’t get the detekt to run as a task for specific module. I followed these steps for the app & submodules I want to apply it on: https://arturbosch.github.io/detekt/gradletask.html when I run

./gradlew :app:detekt

(or

./gradlew :submodule-name:detekt

) I get the following output:

Task 'detekt' not found in project ':app'. Some candidates are: 'test'.

however if I run just

./gradlew detekt

it runs the task.

Is it possible to run detekt only on the files that changed from last commit?

Hi guys. I'm using sonarqube with detekt, but coverage is always 0%. My configuration looks like this:

Hi everyone, I’m sorry if this has been discussed already somewhere. I’m trying to get the benefit of the

ThrowsCount

rule. I have a violated code here:

private fun ensureScanningIsPossible() {
    if (!isDiscoveryPermissionGranted) {
        throw DiscoveryErrors.DiscoveryPermissionsMissingError(neededDiscoveryPermissions)
    } else if (isDiscoveryOngoing) {
        throw DiscoveryErrors.DiscoveryAlreadyStarted
    } else if (!isBluetoothAvailable()) {
        throw DiscoveryErrors.BluetoothDisabledError
    } // More to come later...
}

Why is this discouraged? The doc said: “Functions should have clear 

throw

 statements”. How can we achieve a “clear” version? I tried to refactor by using nullable type but I feel a bit skeptical about the new code:

private fun ensureScanningIsPossible() {
    val error: DiscoveryErrors? = when {
        !isDiscoveryPermissionGranted -> DiscoveryErrors.DiscoveryPermissionsMissingError(neededDiscoveryPermissions)
        isDiscoveryOngoing -> DiscoveryErrors.DiscoveryAlreadyStarted
        !isBluetoothAvailable() -> DiscoveryErrors.BluetoothDisabledError
        else -> null
    }
    error?.let { throw it }
}

How do I configure to disable the ImportWildCard in the gradle?I am running datekBaseline and manually adding it to the blacklist.

Could any of the regular devs have a look at https://github.com/detekt/detekt/pull/2787 and tell me if I’m on the right track or if I am horribly wrong off road? Thanks 😄

Hi All, I am facing strange issue with SpreadOperator as Sonar blocked it https://arturbosch.github.io/detekt/performance.html#spreadoperator . We have gone through this link but we could not understand how to fix this issue if the array is dynamic ( coming from an external library).. Can any one please help me on how to fix this issue.?

Hello everyone, does anyone have the detektAll tasks but in groovy? 😄 https://github.com/sanogueralorenzo/Android-Kotlin-Clean-Architecture/blob/master/build.gradle.kts#L35 I see myself converting to kts everytime just to add the task 😄

I’m having problems using detekt in my project… (kotlin jvm project) I have different problems if I use 1.9.1 or 1.10.0 1.10.0

Execution failed for task ':foo:detekt'.
> Could not resolve all files for configuration ':foo:detekt'.
   > Could not find io.gitlab.arturbosch.detekt:detekt-cli:1.10.0.
     Searched in the following locations:
       - <https://repo.maven.apache.org/maven2/io/gitlab/arturbosch/detekt/detekt-cli/1.10.0/detekt-cli-1.10.0.pom>
     If the artifact you are trying to retrieve can be found in the repository but without metadata in 'Maven POM' format, you need to adjust the 'metadataSources { ... }' of the repository declaration.

I have seen someone mentioning in github issues that it seems that some artifacts weren’t published in all repositories If I try to use 1.9.1:

> Could not find org.jetbrains.kotlinx:kotlinx-html-jvm:0.7.1.
     Searched in the following locations:
       - <https://repo.maven.apache.org/maven2/org/jetbrains/kotlinx/kotlinx-html-jvm/0.7.1/kotlinx-html-jvm-0.7.1.pom>
     If the artifact you are trying to retrieve can be found in the repository but without metadata in 'Maven POM' format, you need to adjust the 'metadataSources { ... }' of the repository declaration.
     Required by:
         project :foo > io.gitlab.arturbosch.detekt:detekt-cli:1.9.1

I’ve tried all possible combinations of the instructions in the website and still haven’t got it to work. Can someone help?

I can't seem to get the

ForbiddenMethodCall

to work, I've got it configured like so:

style:
  active: true
  ForbiddenMethodCall:
    active: true
    methods: ['java.util.concurrent.ExecutorService.submit', 'kotlin.io.println']

in a custom

detekt.yml

(which doesn't contain much else). I'm using the maven plugin and have it pointing to that config file. (I added

println

in there to try and test after the first one wasn't working.) It's not triggering for

println

calls or

ExecutorService.submit

calls. I tried adding another rule and that did work...so maybe I've just got something misconfigured for

ForbiddenMethodCall

?

How can I override settings from a global detekt settings in a module for a multi-module gradle build? Right now I have a

detekt-config.yml

in my root directory, and another `detekt-config.yml`in my module directory. To use both, I use

detekt { config = files("${project.rootDir}/detekt-config.yml", "detekt-config.yml") }

. Is there a better way which e.g. does not require to repeat the base config file name or perhaps even a

mergeConfigs: true

which will automatically merge these? I tried

detekt { config += files("detekt-config.yml") }

but that did not work.

Does detekt have any built-in rules related to inconsistent synchronization? I'm trying to make a case for detekt over spotbugs, but this is a feature we've gotten from spotbugs and haven't found something similar in the detekt docs.

Hi everybody! What’s the way to overwrite the detekt autocorrect flag (defined in

detekt

block in the gradle file) with a command line gradle task parameter? I checked the docs but I didn’t find how. I’ve also tried

./gradlew detekt -Pauto-correct=true

and

./gradlew detekt -PautoCorrect=true

along with some other similar commands but none of them worked.

Hi all. I'm getting this warning:

detekt(DetektExtension.() -> Unit): Unit' is deprecated. Either apply detekt plugin to root project

why? gradle version: 6.5 detekt version: 1.10.

Hi all! Are there any more sophisticated complexity checks that somebody wrote based on Detekt?

ComplexMethod

only works on single methods, but I’m looking for class and inter-class complexity calculation. Is this even possible with Detekt?

Using

detekt

+

ktlint

I'm used to using the Android Studio shortcuts to format a file to fix basic issues. After attempting to upgrade to

1.10.0

this no longer seems to fix the new

Indentation

issue being reported. If a file format command doesn't fix that we can't reasonably upgrade our projects. Any ideas?

Hi. I need help.. There are some code like this

Observable.something()
    .map {
        mutableList.apply {
            addAll(list)
        }
    }

Console says

UnnecessaryApply

.. But I need

return MutableList

I don't know if this is really wrong. How can I fix it? Thank you for reading..

This message was deleted.

Hi, all Does anyone know how to generate a single report on a multi module project? My goal is make Danger (

danger-kotlin_detekt

) read only one file… Thanks in advance

Hi.. I found a simple typo.. https://arturbosch.github.io/detekt/style.html#noncompliant-code-39

a?.let { 1.plus(1) } // can be replaced with `if (a == null) 1.plus(1)`

to

a?:let { 1.plus(1) } // can be replaced with `if (a == null) 1.plus(1)`
a?.let { 1.plus(1) } // can be replaced with `if (a != null) 1.plus(1)`

Hi all, we have a bunch of new releases around detekt [1.11.0](https://github.com/detekt/detekt/releases/tag/v1.11.0): • sonar-detekt [2.2.0](https://github.com/detekt/sonar-kotlin/releases/tag/2.2.0) • intellij-plugin [1.5.0](https://github.com/detekt/detekt-intellij-plugin/releases/tag/v1.5.0) • first public release of the experimental detekt-as-a-compiler-plugin [0.3.0](https://github.com/detekt/detekt-compiler-plugin/tree/v0.3.0) • maven-plugin [1.11.0](https://github.com/Ozsie/detekt-maven-plugin/releases/tag/1.11.0) • github action [1.11.0](https://github.com/marketplace/actions/detekt-all) happy coding!

So I have updated to latest version of IDEA and Detekt IDEA plugin and now get errors about "Unexpected indentation", IDEA format does not fix the problem so I wonder what is causing this

Hello everyone, I was testing out 1.12.0 through cli with the formatting jar too and I noticed this strange formatting after running detekt, is this correct? Looks strange there is no spacing at all after the correction Running with the build-upon-default-config

java -jar config/detekt/detekt-cli-1.12.0-all.jar --build-upon-default-config --includes "**/src/main/**/*.kt" -p config/detekt/detekt-formatting-1.12.0.jar --auto-correct --baseline "config/detekt/baseline.yml" --parallel

I’m seeing lots of results for

ParameterListWrapping

in 1.12.0 that look either questionable or wrong. Avoiding upgrading for now, rather than disabling the rule. A couple of examples: Questionable. Fails with

Argument should be on a separate line (unless all arguments can fit a single line)

and

Missing newline before ")"

.

foo(bar.apply {
    // stuff
})

Wrong? Fails

function(arg1, arg2, arg3)

with

Argument should be on a separate line (unless all arguments can fit a single line)

.

json(
    """
    {
        "array": [
            ${function(arg1, arg2, arg3)}
        ]
    }
    """.trimIndent()
)

Related to this ktlint PR?

[UP] I am having problems with the pre-commit hook. First the changes are commit and after that, I have to commit again to commit the files changed by detekt, anyone is having this issue?

Hello 👋 I was trying to write a pre-commit hook that runs only on changed files (so it takes a second instead of 15 running everything) and I was facing some issues: Some notes: I'm using detekt-cli jar (so I can use the

-input

only on changed files) and also to avoid the gradle overhead/plugin

# Get kotlin changed files
changedFiles=$(git --no-pager diff --diff-filter=d --name-only HEAD | grep '\.kt[s"]\?$')

if [ -n "$changedFiles" ]; then
  # `paste -sd "," -` replaces newlines with commas so detekt cli -i parameter can take it
  java -jar config/detekt/detekt-cli-1.11.2-all.jar --build-upon-default-config --config "config/detekt/detekt.yml" --baseline "config/detekt/baseline.xml" -p config/detekt/detekt-formatting-1.11.2.jar --auto-correct -i "$( printf "%s" "$changedFiles" | paste -sd "," - )"

  if [ $? -ne 0 ]; then
    echo "***********************************************"
    echo "                 Detekt failed                 "
    echo " Please fix the above issues before committing "
    echo "***********************************************"
    exit 1
  fi
fi

The above takes around 2 seconds and it is working properly from the tests I've been doing My main issue is that from

java -jar

to

--auto-correct

is something I already have in my top level detekt task and my top level detekt baseline task (and I wouldn't want to have all that script line repeated in several files since you will have to remember to update all of them when updating version. Is there any way to reuse that part? Thanks! 🙏

There's an open issue