kenkyee
01/02/2020, 6:28 PM**/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 TooManyFunctionsArtur Bosch
01/03/2020, 9:25 PMNamingRules
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("...") }
- 🙂kenkyee
01/03/2020, 10:18 PMtaso
01/06/2020, 2:02 PM