[#119] Detekt 1.22.0

* Bump version

* Update config

buildscript-gradle.lockfile

@@ -6,8 +6,8 @@ com.github.ben-manes:gradle-versions-plugin:0.42.0=classpath
 com.thoughtworks.xstream:xstream:1.4.17=classpath
 io.github.detekt.sarif4k:sarif4k:0.0.1=classpath
 io.github.x-stream:mxparser:1.2.1=classpath
-io.gitlab.arturbosch.detekt:detekt-gradle-plugin:1.21.0=classpath
+io.gitlab.arturbosch.detekt:detekt-gradle-plugin:1.22.0=classpath
-io.gitlab.arturbosch.detekt:io.gitlab.arturbosch.detekt.gradle.plugin:1.21.0=classpath
+io.gitlab.arturbosch.detekt:io.gitlab.arturbosch.detekt.gradle.plugin:1.22.0=classpath
 net.java.dev.jna:jna:5.6.0=classpath
 org.jetbrains.intellij.deps:trove4j:1.0.20200330=classpath
 org.jetbrains.kotlin:kotlin-android-extensions:1.7.21=classpath

gradle.lockfile

@@ -21,29 +21,29 @@ info.picocli:picocli:4.6.3=ktlint
 io.github.davidburstrom.contester:contester-breakpoint:0.2.0=detekt
 io.github.detekt.sarif4k:sarif4k:0.0.1=detekt,ktlint
 io.github.microutils:kotlin-logging-jvm:2.1.23=ktlint
-io.gitlab.arturbosch.detekt:detekt-api:1.21.0=detekt
+io.gitlab.arturbosch.detekt:detekt-api:1.22.0=detekt
-io.gitlab.arturbosch.detekt:detekt-cli:1.21.0=detekt
+io.gitlab.arturbosch.detekt:detekt-cli:1.22.0=detekt
-io.gitlab.arturbosch.detekt:detekt-core:1.21.0=detekt
+io.gitlab.arturbosch.detekt:detekt-core:1.22.0=detekt
-io.gitlab.arturbosch.detekt:detekt-metrics:1.21.0=detekt
+io.gitlab.arturbosch.detekt:detekt-metrics:1.22.0=detekt
-io.gitlab.arturbosch.detekt:detekt-parser:1.21.0=detekt
+io.gitlab.arturbosch.detekt:detekt-parser:1.22.0=detekt
-io.gitlab.arturbosch.detekt:detekt-psi-utils:1.21.0=detekt
+io.gitlab.arturbosch.detekt:detekt-psi-utils:1.22.0=detekt
-io.gitlab.arturbosch.detekt:detekt-report-html:1.21.0=detekt
+io.gitlab.arturbosch.detekt:detekt-report-html:1.22.0=detekt
-io.gitlab.arturbosch.detekt:detekt-report-md:1.21.0=detekt
+io.gitlab.arturbosch.detekt:detekt-report-md:1.22.0=detekt
-io.gitlab.arturbosch.detekt:detekt-report-sarif:1.21.0=detekt
+io.gitlab.arturbosch.detekt:detekt-report-sarif:1.22.0=detekt
-io.gitlab.arturbosch.detekt:detekt-report-txt:1.21.0=detekt
+io.gitlab.arturbosch.detekt:detekt-report-txt:1.22.0=detekt
-io.gitlab.arturbosch.detekt:detekt-report-xml:1.21.0=detekt
+io.gitlab.arturbosch.detekt:detekt-report-xml:1.22.0=detekt
-io.gitlab.arturbosch.detekt:detekt-rules-complexity:1.21.0=detekt
+io.gitlab.arturbosch.detekt:detekt-rules-complexity:1.22.0=detekt
-io.gitlab.arturbosch.detekt:detekt-rules-coroutines:1.21.0=detekt
+io.gitlab.arturbosch.detekt:detekt-rules-coroutines:1.22.0=detekt
-io.gitlab.arturbosch.detekt:detekt-rules-documentation:1.21.0=detekt
+io.gitlab.arturbosch.detekt:detekt-rules-documentation:1.22.0=detekt
-io.gitlab.arturbosch.detekt:detekt-rules-empty:1.21.0=detekt
+io.gitlab.arturbosch.detekt:detekt-rules-empty:1.22.0=detekt
-io.gitlab.arturbosch.detekt:detekt-rules-errorprone:1.21.0=detekt
+io.gitlab.arturbosch.detekt:detekt-rules-errorprone:1.22.0=detekt
-io.gitlab.arturbosch.detekt:detekt-rules-exceptions:1.21.0=detekt
+io.gitlab.arturbosch.detekt:detekt-rules-exceptions:1.22.0=detekt
-io.gitlab.arturbosch.detekt:detekt-rules-naming:1.21.0=detekt
+io.gitlab.arturbosch.detekt:detekt-rules-naming:1.22.0=detekt
-io.gitlab.arturbosch.detekt:detekt-rules-performance:1.21.0=detekt
+io.gitlab.arturbosch.detekt:detekt-rules-performance:1.22.0=detekt
-io.gitlab.arturbosch.detekt:detekt-rules-style:1.21.0=detekt
+io.gitlab.arturbosch.detekt:detekt-rules-style:1.22.0=detekt
-io.gitlab.arturbosch.detekt:detekt-rules:1.21.0=detekt
+io.gitlab.arturbosch.detekt:detekt-rules:1.22.0=detekt
-io.gitlab.arturbosch.detekt:detekt-tooling:1.21.0=detekt
+io.gitlab.arturbosch.detekt:detekt-tooling:1.22.0=detekt
-io.gitlab.arturbosch.detekt:detekt-utils:1.21.0=detekt
+io.gitlab.arturbosch.detekt:detekt-utils:1.22.0=detekt
 net.java.dev.jna:jna:5.6.0=detekt,ktlint
 org.ec4j.core:ec4j-core:0.3.0=ktlint
 org.freemarker:freemarker:2.3.30=IntellijKoverConfig
@@ -55,21 +55,21 @@ org.jetbrains.intellij.deps:coverage-report:1.0.15=IntellijKoverConfig
 org.jetbrains.intellij.deps:intellij-coverage-agent:1.0.668=IntellijKoverConfig
 org.jetbrains.intellij.deps:intellij-coverage-reporter:1.0.668=IntellijKoverConfig
 org.jetbrains.intellij.deps:trove4j:1.0.20200330=detekt,ktlint
-org.jetbrains.kotlin:kotlin-compiler-embeddable:1.6.21=detekt
 org.jetbrains.kotlin:kotlin-compiler-embeddable:1.7.10=ktlint
-org.jetbrains.kotlin:kotlin-daemon-embeddable:1.6.21=detekt
+org.jetbrains.kotlin:kotlin-compiler-embeddable:1.7.21=detekt
 org.jetbrains.kotlin:kotlin-daemon-embeddable:1.7.10=ktlint
-org.jetbrains.kotlin:kotlin-reflect:1.6.21=detekt
+org.jetbrains.kotlin:kotlin-daemon-embeddable:1.7.21=detekt
 org.jetbrains.kotlin:kotlin-reflect:1.7.10=ktlint
-org.jetbrains.kotlin:kotlin-script-runtime:1.6.21=detekt
+org.jetbrains.kotlin:kotlin-reflect:1.7.21=detekt
 org.jetbrains.kotlin:kotlin-script-runtime:1.7.10=ktlint
-org.jetbrains.kotlin:kotlin-stdlib-common:1.6.21=detekt
+org.jetbrains.kotlin:kotlin-script-runtime:1.7.21=detekt
 org.jetbrains.kotlin:kotlin-stdlib-common:1.7.10=ktlint
+org.jetbrains.kotlin:kotlin-stdlib-common:1.7.21=detekt
 org.jetbrains.kotlin:kotlin-stdlib-jdk7:1.7.10=ktlint
 org.jetbrains.kotlin:kotlin-stdlib-jdk8:1.7.10=ktlint
-org.jetbrains.kotlin:kotlin-stdlib:1.6.21=detekt
 org.jetbrains.kotlin:kotlin-stdlib:1.7.10=ktlint
-org.jetbrains.kotlinx:kotlinx-html-jvm:0.7.5=detekt
+org.jetbrains.kotlin:kotlin-stdlib:1.7.21=detekt
+org.jetbrains.kotlinx:kotlinx-html-jvm:0.8.0=detekt
 org.jetbrains.kotlinx:kotlinx-serialization-core-jvm:1.1.0=detekt,ktlint
 org.jetbrains.kotlinx:kotlinx-serialization-core:1.1.0=detekt,ktlint
 org.jetbrains.kotlinx:kotlinx-serialization-json-jvm:1.1.0=detekt,ktlint
@@ -82,5 +82,5 @@ org.ow2.asm:asm-commons:9.2=JacocoKoverConfig
 org.ow2.asm:asm-tree:9.2=JacocoKoverConfig
 org.ow2.asm:asm:9.2=JacocoKoverConfig
 org.slf4j:slf4j-api:2.0.0=ktlint
-org.yaml:snakeyaml:1.30=detekt
+org.yaml:snakeyaml:1.33=detekt
 empty=annotationProcessor,compileClasspath,detektPlugins,runtimeClasspath,testAnnotationProcessor,testCompileClasspath,testRuntimeClasspath

gradle/libs.versions.toml

@@ -11,7 +11,7 @@ kotest-assertion = { module = "io.kotest:kotest-assertions-core", version.ref =
 kotest-property = { module = "io.kotest:kotest-property", version.ref = "kotest" }
 
 [plugins]
-detekt = "io.gitlab.arturbosch.detekt:1.21.0"
+detekt = "io.gitlab.arturbosch.detekt:1.22.0"
 dokka = "org.jetbrains.dokka:1.7.10"
 kover = "org.jetbrains.kotlinx.kover:0.5.1"
 versions = "com.github.ben-manes.versions:0.42.0"

tools/detekt.yml

@@ -10,6 +10,7 @@ build:
 config:
   validation: true
   warningsAsErrors: false
+  checkExhaustiveness: false
   # when writing own rules with new properties, exclude the property path e.g.: 'my_rule_set,.*>.*>[my_property]'
   excludes: ''
 
@@ -78,15 +79,21 @@ comments:
     searchInInnerClass: true
     searchInInnerObject: true
     searchInInnerInterface: true
+    searchInProtectedClass: false
   UndocumentedPublicFunction:
     active: false
     excludes: ['**/test/**', '**/androidTest/**', '**/commonTest/**', '**/jvmTest/**', '**/jsTest/**', '**/iosTest/**']
+    searchProtectedFunction: false
   UndocumentedPublicProperty:
     active: false
     excludes: ['**/test/**', '**/androidTest/**', '**/commonTest/**', '**/jvmTest/**', '**/jsTest/**', '**/iosTest/**']
+    searchProtectedProperty: false
 
 complexity:
   active: true
+  CognitiveComplexMethod:
+    active: false
+    threshold: 15
   ComplexCondition:
     active: true
     threshold: 4
@@ -95,7 +102,8 @@ complexity:
     threshold: 10
     includeStaticDeclarations: false
     includePrivateDeclarations: false
-  ComplexMethod:
+    ignoreOverloaded: false
+  CyclomaticComplexMethod:
     active: true
     threshold: 15
     ignoreSingleWhenExpression: false
@@ -386,6 +394,8 @@ performance:
   SpreadOperator:
     active: true
     excludes: ['**/test/**', '**/androidTest/**', '**/commonTest/**', '**/jvmTest/**', '**/jsTest/**', '**/iosTest/**']
+  UnnecessaryPartOfBinaryExpression:
+    active: false
   UnnecessaryTemporaryInstantiation:
     active: true
 
@@ -412,8 +422,6 @@ potential-bugs:
       - 'java.util.HashSet'
       - 'java.util.LinkedHashMap'
       - 'java.util.HashMap'
-  DuplicateCaseInWhenExpression:
-    active: true
   ElseCaseInsteadOfExhaustiveWhen:
     active: false
   EqualsAlwaysReturnsTrueOrFalse:
@@ -428,12 +436,16 @@ potential-bugs:
     active: true
   IgnoredReturnValue:
     active: true
-    restrictToAnnotatedMethods: true
+    restrictToConfig: true
     returnValueAnnotations:
       - '*.CheckResult'
       - '*.CheckReturnValue'
     ignoreReturnValueAnnotations:
       - '*.CanIgnoreReturnValue'
+    returnValueTypes:
+      - 'kotlin.sequences.Sequence'
+      - 'kotlinx.coroutines.flow.*Flow'
+      - 'java.util.stream.*Stream'
     ignoreFunctionCall: []
   ImplicitDefaultLocale:
     active: true
@@ -455,17 +467,14 @@ potential-bugs:
   MissingPackageDeclaration:
     active: false
     excludes: ['**/*.kts']
-  MissingWhenCase:
-    active: true
-    allowElseExpression: true
   NullCheckOnMutableProperty:
     active: false
   NullableToStringCall:
     active: false
-  RedundantElseInWhen:
-    active: true
   UnconditionalJumpStatementInLoop:
     active: false
+  UnnecessaryNotNullCheck:
+    active: false
   UnnecessaryNotNullOperator:
     active: true
   UnnecessarySafeCall:
@@ -488,6 +497,8 @@ potential-bugs:
 
 style:
   active: true
+  AlsoCouldBeApply:
+    active: false
   CanBeNonNullable:
     active: false
   CascadingCallWrapping:
@@ -499,7 +510,8 @@ style:
     active: false
   DataClassContainsFunctions:
     active: false
-    conversionFunctionPrefix: 'to'
+    conversionFunctionPrefix:
+      - 'to'
   DataClassShouldBeImmutable:
     active: false
   DestructuringDeclarationWithTooManyEntries:
@@ -531,14 +543,10 @@ style:
   ForbiddenMethodCall:
     active: false
     methods:
-      - 'kotlin.io.print'
+      - reason: 'print does not allow you to configure the output stream. Use a logger instead.'
-      - 'kotlin.io.println'
+        value: 'kotlin.io.print'
-  ForbiddenPublicDataClass:
+      - reason: 'println does not allow you to configure the output stream. Use a logger instead.'
-    active: true
+        value: 'kotlin.io.println'
-    excludes: ['**']
-    ignorePackages:
-      - '*.internal'
-      - '*.internal.*'
   ForbiddenSuppress:
     active: false
     rules: []
@@ -550,13 +558,7 @@ style:
     active: true
     ignoreOverridableFunction: true
     ignoreActualFunction: true
-    excludedFunctions: ''
+    excludedFunctions: []
-  LibraryCodeMustSpecifyReturnType:
-    active: true
-    excludes: ['**']
-  LibraryEntitiesShouldNotBePublic:
-    active: true
-    excludes: ['**']
   LoopWithTooManyJumpStatements:
     active: true
     maxJumpCount: 1
@@ -591,16 +593,20 @@ style:
     excludePackageStatements: true
     excludeImportStatements: true
     excludeCommentStatements: false
+    excludeRawStrings: true
   MayBeConst:
     active: true
   ModifierOrder:
     active: true
   MultilineLambdaItParameter:
     active: false
+  MultilineRawStringIndentation:
+    active: false
+    indentSize: 4
   NestedClassesVisibility:
     active: true
   NewLineAtEndOfFile:
-    active: false
+    active: true
   NoTabs:
     active: false
   NullableBooleanCheck:
@@ -626,7 +632,8 @@ style:
   ReturnCount:
     active: true
     max: 2
-    excludedFunctions: 'equals'
+    excludedFunctions:
+      - 'equals'
     excludeLabeled: false
     excludeReturnFromLambda: true
     excludeGuardClauses: false
@@ -642,6 +649,8 @@ style:
     excludeGuardClauses: false
   TrailingWhitespace:
     active: false
+  TrimMultilineRawString:
+    active: false
   UnderscoresInNumericLiterals:
     active: false
     acceptableLength: 4
@@ -664,6 +673,7 @@ style:
     active: false
   UnnecessaryParentheses:
     active: false
+    allowForUnclearPrecedence: false
   UntilInsteadOfRangeTo:
     active: false
   UnusedImports:
@@ -698,6 +708,8 @@ style:
     active: true
   UseRequireNotNull:
     active: true
+  UseSumOfInsteadOfFlatMapSize:
+    active: false
   UselessCallOnNotNull:
     active: true
   UtilityClassWithPublicConstructor:
@@ -707,6 +719,5 @@ style:
     ignoreLateinitVar: false
   WildcardImport:
     active: true
-    excludes: ['**/test/**', '**/androidTest/**', '**/commonTest/**', '**/jvmTest/**', '**/jsTest/**', '**/iosTest/**']
     excludeImports:
       - 'java.util.*'