"are now configured to exclude the

**/test/**,**/*Test.kt,**/*Spec.kt"

folder (this is valid only if you were using the default config file)" ahh...FWIW, it was nicer having the excludes in one place since now it means you need to duplicate the excludes glob match list in a bunch of places. We did it for comments and these: - 'NamingRules' - 'WildcardImport' - 'MagicNumber' - 'MaxLineLength' - 'LateinitUsage' - 'StringLiteralDuplication' - 'SpreadOperator' - 'TooManyFunctions' - 'ForEachOnRange' - 'FunctionMaxLength' Looks like the default only does comments and: StringLiteralDuplication and TooManyFunctions

Yep, it got a bit more verbose but is more flexible now and allow api-only rules for example. FYI

NamingRules

got deprecated in favor of the

naming

rule set.

excludes

and

includes

do work on the rule set level. In the long run when the scripting support of Kotlin gets a lot faster, we could introduce way to configure detekt with a kotlin script and get the compact wau to defines excludes back -

setOf("WildcardImport", ...).forEach { it.excludes("...") }

- 🙂

sounds good...that would be nice 🙂

Just checked the default config in the repo, it looks like more of these are excluded for tests.