ghost
commented
1 year ago 👇 Click on the image for a new way to code review
#### [](https://app.codesee.io/r/reviews?pr=358&src=https%3A%2F%2Fgithub.com%2Fmallowigi%2Fa-file-icon-idea) #### Legend
This PR contains the following updates:
1.22.0->1.23.011.3.1->11.5.01.22.0->1.23.02.0.0->2.1.11.13.3->1.15.01.8.10->1.9.0Release Notes
detekt/detekt (io.gitlab.arturbosch.detekt:detekt-formatting)
### [`v1.23.0`](https://togithub.com/detekt/detekt/releases/tag/v1.23.0) We're so proud to announce the next upcoming stable release of detekt: 1.23.0 🚀 This release is coming with 25 new rules and much more. We moved the detekt-compiler-plugin to the main repo, and you can now use it in your builds [following the official documentation](https://detekt.dev/docs/gettingstarted/compilerplugin). This release also marks the beginning of development of 2.x from the main branch! [You can follow the progress here](https://togithub.com/detekt/detekt/discussions/6035) We want to take the opportunity to thank [our Sponsors](https://togithub.com/sponsors/detekt) and our Contributors for testing, bug reporting and helping us release this new version of Detekt. You're more than welcome to join our community on the [#detekt](https://kotlinlang.slack.com/archives/C88E12QH4) channel on KotlinLang's Slack (you can [get an invite here](https://surveys.jetbrains.com/s3/kotlin-slack-sign-up)). ##### Notable Changes - This is the first version of Detekt that ships with the `detekt-compiler-plugin`. The [Detekt Compiler plugin](https://detekt.dev/docs/gettingstarted/compilerplugin) is still experimental, but we're moving it closer to Detekt to make it easier to integrate. From now on the compiler plugin will follow the same versioning schema as Detekt. We invite you to try it and provide feedback till we stabilize it. You can read more about it in the [official documentation page](https://detekt.dev/docs/gettingstarted/compilerplugin) - [#5492](https://togithub.com/detekt/detekt/pull/5492) - We added **25** new Rules to detekt - `BracesOnIfStatements` - [#5700](https://togithub.com/detekt/detekt/pull/5700) - `BracesOnWhenStatements` - [#5838](https://togithub.com/detekt/detekt/pull/5838) - `CastNullableToNonNullableType` - [#5653](https://togithub.com/detekt/detekt/pull/5653) - `DoubleNegativeLambda` - [#5937](https://togithub.com/detekt/detekt/pull/5937) - `ForbiddenAnnotation` - [#5515](https://togithub.com/detekt/detekt/pull/5515) - `PropertyUsedBeforeDeclaration` - [#6062](https://togithub.com/detekt/detekt/pull/6062) - `StringShouldBeRawString` - [#5705](https://togithub.com/detekt/detekt/pull/5705) - `SuspendFunSwallowedCancellation` - [#5666](https://togithub.com/detekt/detekt/pull/5666) - `UnusedParameter` - [#5722](https://togithub.com/detekt/detekt/pull/5722) - `UnusedPrivateProperty` - [#5722](https://togithub.com/detekt/detekt/pull/5722) - `UseLet` - [#6091](https://togithub.com/detekt/detekt/pull/6091) - `UnnecessaryBracesAroundTrailingLambda` - [#6029](https://togithub.com/detekt/detekt/pull/6029) - Plus the bump to KtLint 0.49.1 added the following rules to the `detekt-formatting` ruleset: - `ClassName` - [#6037](https://togithub.com/detekt/detekt/pull/6037) - `EnumWrapping` - [#6028](https://togithub.com/detekt/detekt/pull/6028) - `FunctionName` - [#6037](https://togithub.com/detekt/detekt/pull/6037) - `IfElseBracing` - [#6028](https://togithub.com/detekt/detekt/pull/6028) - `IfElseWrapping` - [#6028](https://togithub.com/detekt/detekt/pull/6028) - `MultilineExpressionWrapping` - [#6028](https://togithub.com/detekt/detekt/pull/6028) - `NoBlankLineInList` - [#6028](https://togithub.com/detekt/detekt/pull/6028) - `NoConsecutiveComments` - [#6028](https://togithub.com/detekt/detekt/pull/6028) - `NoEmptyFirstLineInClassBody` - [#6028](https://togithub.com/detekt/detekt/pull/6028) - `NoSingleLineBlockCommentRule` - [#6104](https://togithub.com/detekt/detekt/pull/6104) - `ParameterWrapping` - [#6028](https://togithub.com/detekt/detekt/pull/6028) - `PropertyName` - [#6037](https://togithub.com/detekt/detekt/pull/6037) - `PropertyWrapping` - [#6028](https://togithub.com/detekt/detekt/pull/6028) - `StringTemplateIndent` - [#6028](https://togithub.com/detekt/detekt/pull/6028) - `TryCatchFinallySpacing` - [#6028](https://togithub.com/detekt/detekt/pull/6028) - Notable changes to existing rules: - `UnnecessaryAbstractClass` now only runs with type resolution - [#5829](https://togithub.com/detekt/detekt/pull/5829) - `UnusedPrivateMember` has been refactored with some of its logic moved to `UnusedParameter` and `UnusedPrivateProperty` - [#5722](https://togithub.com/detekt/detekt/pull/5722) - Removed the `ignoreOverridden` config from `BooleanPropertyNaming`, `ConstructorParameterNaming`, `FunctionNaming`, `VariableNaming` and `FunctionParameterNaming` as not useful for those rules - [#5718](https://togithub.com/detekt/detekt/pull/5718) - Added `ignoredSubjectTypes` to rule `ElseCaseInsteadOfExhaustiveWhen` to specify types that should be ignored by the rule ([#5623](https://togithub.com/detekt/detekt/issues/5623)) - [#5634](https://togithub.com/detekt/detekt/pull/5634) - Added `allowOperators` to rule `DataClassContainsFunctions` - [#5658](https://togithub.com/detekt/detekt/pull/5658) - `MandatoryBracesIfStatements` has been removed in favor of `BracesOnIfStatements` - [#5700](https://togithub.com/detekt/detekt/pull/5700) - Added `ignoreWhenContainingVariableDeclaration` to `UseIfInsteadOfWhen` to ignore captured variables - [#5681](https://togithub.com/detekt/detekt/pull/5681) - Several rules in the `detekt-formatting` ruleset now accepts an `indentSize` parameter (see the [full list here](https://togithub.com/detekt/detekt/pull/6028/files#diff-2486d56e6f1bbfcb160b923d15266691f5776948cf5bb31c8fc102bd3cf9513d)). - We followed the KtLint decisions on turning on some rules by default, so several rules in the `detekt-formatting` ruleset are now turned on by default (see the [full list here](https://togithub.com/detekt/detekt/pull/6028/files#diff-2486d56e6f1bbfcb160b923d15266691f5776948cf5bb31c8fc102bd3cf9513d)). - We added support for Gradle's Worker API inside Detekt Gradle Plugin, for faster execution on bigger projects. - [#4128](https://togithub.com/detekt/detekt/pull/4128) - We fixed the `includes`/`excludes` logic on the config file as they were overriding each other - [#5782](https://togithub.com/detekt/detekt/pull/5782) - We fully removed support for Spek from `detekt-test-utils`. The recommended testing framework is JUnit - [#5785](https://togithub.com/detekt/detekt/pull/5785) - The minimum supported Gradle version is now `v6.8.3` - [#5616](https://togithub.com/detekt/detekt/pull/5616) - This version of detekt is built with Gradle `v8.1`, AGP `8.0.1`, Kotlin `1.8.21` and KtLint `0.49.1` (see [#5893](https://togithub.com/detekt/detekt/pull/5893) [#5723](https://togithub.com/detekt/detekt/pull/5723) [#5877](https://togithub.com/detekt/detekt/pull/5877) [#6028](https://togithub.com/detekt/detekt/pull/6028) [#6043](https://togithub.com/detekt/detekt/pull/6043) [#5995](https://togithub.com/detekt/detekt/pull/5995) [#5996](https://togithub.com/detekt/detekt/pull/5996)) - We now added a [Code of Conduct](https://togithub.com/detekt/detekt/blob/main/.github/CODE_OF_CONDUCT.md) to our repo. Please read it and follow it when interacting with our community on our channels. ##### Changelog - SerialVersionUIDInSerializableClass - Update the error location - [#6114](https://togithub.com/detekt/detekt/pull/6114) - Reduce LoopWithTooManyJumpStatements finding scope - [#6110](https://togithub.com/detekt/detekt/pull/6110) - Add alias for SuspendFunWithCoroutineScopeReceiver - [#6089](https://togithub.com/detekt/detekt/pull/6089) - CastNullableToNonNullableType - Check the SimpleType instead of typeElement - [#6071](https://togithub.com/detekt/detekt/pull/6071) - Update plugin com.gradle.enterprise to v3.13.1 - [#6069](https://togithub.com/detekt/detekt/pull/6069) - CanBeNonNullable: Check parent condition for checking if nullability info is used or not - [#6064](https://togithub.com/detekt/detekt/pull/6064) - Add configuration to add alternate trimming methods - [#6063](https://togithub.com/detekt/detekt/pull/6063) - Check if property is documented at class header - [#6061](https://togithub.com/detekt/detekt/pull/6061) - OutdatedDocumentation - Check if only public property is documented - [#6057](https://togithub.com/detekt/detekt/pull/6057) - UnnecessaryLet: fix false positive in call chains - [#6052](https://togithub.com/detekt/detekt/pull/6052) - Add `comments` with a list of regexes to `ForbiddenComment` - [#5981](https://togithub.com/detekt/detekt/pull/5981) - Fix incomplete `requireRootInDeclaration` check in `InvalidPackageDeclaration` - [#6045](https://togithub.com/detekt/detekt/pull/6045) - BracesOnWhenStatements: fix false positive for necessary braces - [#6042](https://togithub.com/detekt/detekt/pull/6042) - Fix redundant ClassOrdering violations using maximum increasing section - [#6003](https://togithub.com/detekt/detekt/pull/6003) - UseIsNullOrEmpty: fix false negative with chained call - [#6027](https://togithub.com/detekt/detekt/pull/6027) - Create docs for Gradle Worker API - [#6016](https://togithub.com/detekt/detekt/pull/6016) - Compile compiler plugin against kotlin-compiler-embeddable - [#6012](https://togithub.com/detekt/detekt/pull/6012) - Update intro.mdx to use setFrom() on detekt config - [#6010](https://togithub.com/detekt/detekt/pull/6010) - Use `detekt-versions.properties` instead of `versions.properties` - [#6006](https://togithub.com/detekt/detekt/pull/6006) - Implement parentPath accessor in concrete config implementations - [#6002](https://togithub.com/detekt/detekt/pull/6002) - Increase ALIASES_LIMIT to 100 for YamlConfig - [#5986](https://togithub.com/detekt/detekt/pull/5986) - Fix elvis to +/- case in case UnnecessaryParentheses when precedence is unclear - [#5983](https://togithub.com/detekt/detekt/pull/5983) - jdkHome as [@Internal](https://togithub.com/Internal) - [#5978](https://togithub.com/detekt/detekt/pull/5978) - Fix broken POM file for detetk-compiler-plugin - [#5971](https://togithub.com/detekt/detekt/pull/5971) - Fix broken publishToMavenLocal - [#5970](https://togithub.com/detekt/detekt/pull/5970) - Add `allowOmitUnit` to rule `LibraryCodeMustSpecifyReturnType` - [#5861](https://togithub.com/detekt/detekt/pull/5861) - Remove dependency that creates a cycle. - [#5777](https://togithub.com/detekt/detekt/pull/5777) - Update dependency org.jetbrains:annotations to v24 - [#5969](https://togithub.com/detekt/detekt/pull/5969) - Update github/codeql-action digest to [`8c8d71d`](https://togithub.com/detekt/detekt/commit/8c8d71d) - [#5966](https://togithub.com/detekt/detekt/pull/5966) - Add functions to ExitOutsideMain rule - [#5963](https://togithub.com/detekt/detekt/pull/5963) - Update README.md - [#5954](https://togithub.com/detekt/detekt/pull/5954) - Prevent import statements from counting as references for UnusedPrivateProperty - [#5942](https://togithub.com/detekt/detekt/pull/5942) - Fix ExpressionBodySyntax not checking property getters/setters - [#5938](https://togithub.com/detekt/detekt/pull/5938) - Improve correctness of UnusedPrivateProperty - [#5935](https://togithub.com/detekt/detekt/pull/5935) - Fix documentation for deprecated 'reports' object (Issue [#5908](https://togithub.com/detekt/detekt/issues/5908)) - [#5924](https://togithub.com/detekt/detekt/pull/5924) - Print file path report as link file - [#5921](https://togithub.com/detekt/detekt/pull/5921) - "detekt" or "Detekt" - [#5898](https://togithub.com/detekt/detekt/issues/5898) - Update JSON schema URL - [#5881](https://togithub.com/detekt/detekt/pull/5881) - Add support for local suppression inside formatting - [#5876](https://togithub.com/detekt/detekt/pull/5876) - Fix checkExhaustiveness for formatting and third party rules - [#5869](https://togithub.com/detekt/detekt/pull/5869) - Allow newline style for MaxChainedCallsOnSameLine - [#5865](https://togithub.com/detekt/detekt/pull/5865) - Declare inputs and outputs to support incremental build for testPluginKotlinc - [#5862](https://togithub.com/detekt/detekt/pull/5862) - Use code syntax for `Unit` for ImplicitUnitReturnType rule - [#5857](https://togithub.com/detekt/detekt/pull/5857) - MatchingDeclarationName. KtFilesSpec also remove .common.kt suffix from kotlin files - [#5851](https://togithub.com/detekt/detekt/pull/5851) - Fix double mutability issues with Gradle plugin's use of ConfigurableFileCollection - [#5850](https://togithub.com/detekt/detekt/pull/5850) - MagicNumber - Make ignoreNamedArgument catch more complex expression - [#5837](https://togithub.com/detekt/detekt/pull/5837) - Exclude super call when generating guard clauses - [#5835](https://togithub.com/detekt/detekt/pull/5835) - Enable test retry for all our modules - [#5825](https://togithub.com/detekt/detekt/pull/5825) - Stop configuring report merge tasks while configuring Detekt tasks - [#5813](https://togithub.com/detekt/detekt/pull/5813) - FunctionMatcher support for fully qualified function names - [#5812](https://togithub.com/detekt/detekt/pull/5812) - Boy Scout - [#5808](https://togithub.com/detekt/detekt/pull/5808) - Simplify `TestConfig` usages - [#5801](https://togithub.com/detekt/detekt/pull/5801) - Reduce configuration of UnusedPrivateMember's split rules - [#5800](https://togithub.com/detekt/detekt/pull/5800) - Force SerialVerionUID to be private - [#5798](https://togithub.com/detekt/detekt/pull/5798) - Exclude the KMP test folders for android - [#5797](https://togithub.com/detekt/detekt/pull/5797) - Add aliases for PackageNaming and InvalidPackageDeclaration - [#5795](https://togithub.com/detekt/detekt/pull/5795) - Forbid using Jupiter Kotlin assertions - [#5794](https://togithub.com/detekt/detekt/pull/5794) - ModifierOrder: fix false positive with block comments - [#5791](https://togithub.com/detekt/detekt/pull/5791) - Fixed ProtectedMemberInFinalClass rule reporting valid JVM finalize - [#5788](https://togithub.com/detekt/detekt/pull/5788) - Remove unnecessary usage of BeforeAll in tests - [#5781](https://togithub.com/detekt/detekt/pull/5781) - Run Kotlin compiler plugin CLI test as part of standard build - [#5766](https://togithub.com/detekt/detekt/pull/5766) - Refactor to remove java.util.Array\* imports - [#5761](https://togithub.com/detekt/detekt/pull/5761) - Replace java.util.Stack with Kotlin's ArrayDeque implementation - [#5760](https://togithub.com/detekt/detekt/pull/5760) - Replace java.time.Duration with kotlin.time.Duration - [#5759](https://togithub.com/detekt/detekt/pull/5759) - Add NixOS installation method to doc - [#5757](https://togithub.com/detekt/detekt/pull/5757) - Remove & forbid usage of java.util.stream - [#5756](https://togithub.com/detekt/detekt/pull/5756) - Use stdlib functions for file & path operations - [#5754](https://togithub.com/detekt/detekt/pull/5754) - Enable UnnecessaryBackticks in detekt project - [#5753](https://togithub.com/detekt/detekt/pull/5753) - Update docusaurus monorepo to v2.3.1 - [#5752](https://togithub.com/detekt/detekt/pull/5752) - Resolve runtime classpaths consistently with compile classpaths - [#5730](https://togithub.com/detekt/detekt/pull/5730) - Exclude new Kotlin Test directories in default config - [#5727](https://togithub.com/detekt/detekt/issues/5727) - Fail when unexpected version of kotlin-compiler-embeddable is on runtime classpath - [#5726](https://togithub.com/detekt/detekt/pull/5726) - Fix IgnoredReturnValue rule crash in parallel mode - [#5724](https://togithub.com/detekt/detekt/pull/5724) - Use `name` that return name after backticks - [#5719](https://togithub.com/detekt/detekt/pull/5719) - UnusedPrivateMember - Fix false positive in case of invoke operator - [#5717](https://togithub.com/detekt/detekt/pull/5717) - Fix false positive for `CanBeNonNullable` rule - [#5714](https://togithub.com/detekt/detekt/pull/5714) - IgnoredReturnValue: fix false negative when annotation is on the package - [#5706](https://togithub.com/detekt/detekt/pull/5706) - Check Thread.sleep for block expression - [#5699](https://togithub.com/detekt/detekt/pull/5699) - Fix false positive of in UnnecessaryParentheses - [#5684](https://togithub.com/detekt/detekt/pull/5684) - Fix url and kotlin reference support in kdoc - [#5683](https://togithub.com/detekt/detekt/pull/5683) - Add config for variable expression in when - [#5681](https://togithub.com/detekt/detekt/pull/5681) - Enable NoSemicolons rule - [#5663](https://togithub.com/detekt/detekt/pull/5663) - Use correct resolvable/consumable flags on detekt's configurations - [#5657](https://togithub.com/detekt/detekt/pull/5657) - Prepare for Gradle 8 - [#5656](https://togithub.com/detekt/detekt/pull/5656) - ExplicitCollectionElementAccessMethod rule update - [#5654](https://togithub.com/detekt/detekt/pull/5654) - Cast nullable to non nullable type - [#5653](https://togithub.com/detekt/detekt/pull/5653) - Enable verbose mode for Codecov GH Action - [#5652](https://togithub.com/detekt/detekt/pull/5652) - Fail build when issues found with JVM target compatibility of related compile tasks - [#5651](https://togithub.com/detekt/detekt/pull/5651) - Don't silently use Kotlin compiler fallback strategy - [#5650](https://togithub.com/detekt/detekt/pull/5650) - Documentation tweaks - [#5639](https://togithub.com/detekt/detekt/pull/5639) - Have consistent compile-test-snippets between project and system property - [#5630](https://togithub.com/detekt/detekt/pull/5630) - Exclude operator functions in function min/max length - [#5618](https://togithub.com/detekt/detekt/pull/5618) - Broken link at EmptyFunctionBlock - [#5604](https://togithub.com/detekt/detekt/pull/5604) - Update rule description for errorprone rules - [#5603](https://togithub.com/detekt/detekt/pull/5603) - FunctionMaxLength false positive for overridden methods [#5590](https://togithub.com/detekt/detekt/issues/5590) - [#5599](https://togithub.com/detekt/detekt/pull/5599) - Report proper code position in `MaxLineLength` - [#5583](https://togithub.com/detekt/detekt/pull/5583) - Allow access to nonpublic members of public types in java.base/java.lang package - [#5579](https://togithub.com/detekt/detekt/pull/5579) - NonBooleanPropertyPrefixedWithIs: Allow AtomicBoolean - [#5577](https://togithub.com/detekt/detekt/pull/5577) - Pass compilation output classes to detekt analysis classpath - [#5556](https://togithub.com/detekt/detekt/pull/5556) - Fix KDocReferencesNonPublicProperty false positive - [#5534](https://togithub.com/detekt/detekt/pull/5534) ##### Dependency Updates - Update dependency io.github.detekt.sarif4k:sarif4k to v0.4.0 - [#6113](https://togithub.com/detekt/detekt/pull/6113) - Update dependency org.jetbrains.kotlinx:kotlinx-coroutines-core to v1.7.1 - [#6097](https://togithub.com/detekt/detekt/pull/6097) - Update dependency org.jetbrains.kotlinx:kotlinx-coroutines-core to v1.7.0 - [#6074](https://togithub.com/detekt/detekt/pull/6074) - Update com.android.tools.build - [#6065](https://togithub.com/detekt/detekt/pull/6065) - Update JaCoCo to v0.8.10 - [#6044](https://togithub.com/detekt/detekt/pull/6044) - Update plugin pluginPublishing to v1.2.0 - [#5975](https://togithub.com/detekt/detekt/pull/5975) - Update ktlint to v0.48.1 - [#5661](https://togithub.com/detekt/detekt/pull/5661) - Update dependency com.android.tools.build:gradle to v7.4.0 - [#5693](https://togithub.com/detekt/detekt/pull/5693) - JaCoCo 0.8.9 - [#5959](https://togithub.com/detekt/detekt/pull/5959) - Update dependency com.github.tschuchortdev:kotlin-compile-testing to v1.5.0 - [#5882](https://togithub.com/detekt/detekt/pull/5882) - Update dependency org.jetbrains.dokka to v1.8.10 - [#5878](https://togithub.com/detekt/detekt/pull/5878) - Compile detekt-compiler-plugin against standard Kotlin compiler artifact - [#5765](https://togithub.com/detekt/detekt/pull/5765) - Migrate to SnakeYAML Engine - [#5751](https://togithub.com/detekt/detekt/pull/5751) - Update kotlin monorepo to v1.8.10 - [#5745](https://togithub.com/detekt/detekt/pull/5745) - Update ktlint to v0.48.0 - [#5625](https://togithub.com/detekt/detekt/pull/5625) - Migrate to Gradle Nexus Publish Plugin - [#5554](https://togithub.com/detekt/detekt/pull/5554) - Update dependency io.github.detekt.sarif4k:sarif4k to v0.2.0 - [#5496](https://togithub.com/detekt/detekt/pull/5496) ##### Housekeeping & Refactorings - Inline Cases enum and inline other external test code into the test classes - [#6107](https://togithub.com/detekt/detekt/pull/6107) - Update codecov/codecov-action digest to [`eaaf4be`](https://togithub.com/detekt/detekt/commit/eaaf4be) - [#6102](https://togithub.com/detekt/detekt/pull/6102) - Remove unnecessary baselines - [#6092](https://togithub.com/detekt/detekt/pull/6092) - Remove unused `dependenciesAsNames` - [#6059](https://togithub.com/detekt/detekt/pull/6059) - Reduce eager POM task creation - [#6041](https://togithub.com/detekt/detekt/pull/6041) - Improve our configuration of `ClassNaming` and `FunctionNaming` - [#6019](https://togithub.com/detekt/detekt/pull/6019) - Comment text in the Issue/PR Template - [#5992](https://togithub.com/detekt/detekt/pull/5992) - Fix typo: Github -> GitHub - [#5956](https://togithub.com/detekt/detekt/pull/5956) - Review all Detekt and Detekt Gradle Plugin usages. - [#5955](https://togithub.com/detekt/detekt/pull/5955) - Review all Detekt and Detekt Gradle Plugin usages. - [#5953](https://togithub.com/detekt/detekt/pull/5953) - Add a test for catching undocumented public interfaces - [#5951](https://togithub.com/detekt/detekt/pull/5951) - Execute tests in parallel - [#5944](https://togithub.com/detekt/detekt/pull/5944) - Make GeneratorSpec use resources - [#5932](https://togithub.com/detekt/detekt/pull/5932) - Cleanup detekt-formatting to use detekt's own assertThat function - [#5911](https://togithub.com/detekt/detekt/pull/5911) - Stale any issue with support tag in one month - [#5904](https://togithub.com/detekt/detekt/pull/5904) - Remove instances of double mutability - [#5899](https://togithub.com/detekt/detekt/pull/5899) - Handle todo in LinesOfCode logic - [#5897](https://togithub.com/detekt/detekt/pull/5897) - Boy scout - `detekt-generator` - [#5854](https://togithub.com/detekt/detekt/pull/5854) - Tweak GHA configs a bit - [#5852](https://togithub.com/detekt/detekt/pull/5852) - Create `generateWebsite` - [#5849](https://togithub.com/detekt/detekt/pull/5849) - Remove dependency between check and jacocoMergedReport - [#5846](https://togithub.com/detekt/detekt/pull/5846) - Sort deprecation properties - [#5845](https://togithub.com/detekt/detekt/pull/5845) - Simplify generate documentation - [#5844](https://togithub.com/detekt/detekt/pull/5844) - Remove unnecessary `@Suppress("ReturnCount")` - [#5841](https://togithub.com/detekt/detekt/pull/5841) - Cleaner merging of Gradle blocks for functionalTests - [#5830](https://togithub.com/detekt/detekt/pull/5830) - Replace trimMargin usages - [#5827](https://togithub.com/detekt/detekt/pull/5827) - Disable PTS from local and enable it for PRs - [#5826](https://togithub.com/detekt/detekt/pull/5826) - Activate MultilineRawStringIndentation on detekt - [#5819](https://togithub.com/detekt/detekt/pull/5819) - Remove single-use `times` method, use stdlib's `repeat` instead. - [#5774](https://togithub.com/detekt/detekt/pull/5774) - Remove redundant build config - [#5617](https://togithub.com/detekt/detekt/pull/5617) - Migrate to AGP namespaces - [#5569](https://togithub.com/detekt/detekt/pull/5569) - Fix typo - [#5557](https://togithub.com/detekt/detekt/pull/5557) ##### Contributors We would like to thank the following contributors that made this release possible: [@3flex](https://togithub.com/3flex), [@BeBAKE](https://togithub.com/BeBAKE), [@BraisGabin](https://togithub.com/BraisGabin), [@Goooler](https://togithub.com/Goooler), [@SaumyaBhushan](https://togithub.com/SaumyaBhushan), [@TWiStErRob](https://togithub.com/TWiStErRob), [@VitalyVPinchuk](https://togithub.com/VitalyVPinchuk), [@adef145](https://togithub.com/adef145), [@asomov](https://togithub.com/asomov), [@atulgpt](https://togithub.com/atulgpt), [@chao2zhang](https://togithub.com/chao2zhang), [@cketti](https://togithub.com/cketti), [@cortinico](https://togithub.com/cortinico), [@drawers](https://togithub.com/drawers), [@dzirbel](https://togithub.com/dzirbel), [@igorwojda](https://togithub.com/igorwojda), [@lexa-diky](https://togithub.com/lexa-diky), [@luanpotter](https://togithub.com/luanpotter), [@marschwar](https://togithub.com/marschwar), [@mjovanc](https://togithub.com/mjovanc), [@mmorozkov](https://togithub.com/mmorozkov), [@ncteisen](https://togithub.com/ncteisen), [@osipxd](https://togithub.com/osipxd), [@ov7a](https://togithub.com/ov7a), [@schalkms](https://togithub.com/schalkms), [@t-kameyama](https://togithub.com/t-kameyama), [@tresni](https://togithub.com/tresni) See all issues at: [1.23.0](https://togithub.com/detekt/detekt/milestone/88)JetBrains/kotlin (org.jetbrains.kotlin.jvm)
### [`v1.9.0`](https://togithub.com/JetBrains/kotlin/releases/tag/v1.9.0): Kotlin 1.9.0 ##### Changelog ##### Analysis API ##### New Features - [`KT-57930`](https://youtrack.jetbrains.com/issue/KT-57930) Analysis API: provide an API for extending Kotlin resolution - [`KT-57636`](https://youtrack.jetbrains.com/issue/KT-57636) K2: Add the return type of K2 reference shortener AA `ShortenCommand::invokeShortening()` e.g., `ShorteningResultInfo` to allow callers to access the shortening result PSI ##### Fixes - [`KT-58249`](https://youtrack.jetbrains.com/issue/KT-58249) Analysis API: Disable error logging for FE10 implementation of resolveCall when resolve is not successful - [`KT-55626`](https://youtrack.jetbrains.com/issue/KT-55626) Impossible to restore symbol by psi from script file - [`KT-57314`](https://youtrack.jetbrains.com/issue/KT-57314) LL FIR: Combine `LLFirProvider$SymbolProvider`s in session dependencies (optimization) - [`KT-55527`](https://youtrack.jetbrains.com/issue/KT-55527) K2 IDE: Rewrite KtScopeContext class to allow to handle each scope separately - [`KT-55329`](https://youtrack.jetbrains.com/issue/KT-55329) LL FIR: Unexpected ACTUAL_WITHOUT_EXPECT error on constructor and function declaration - [`KT-50732`](https://youtrack.jetbrains.com/issue/KT-50732) LL API: fix compiler based tests - [`KT-57850`](https://youtrack.jetbrains.com/issue/KT-57850) K2: contract violation due to SymbolLightAccessorMethod.propertyAccessorSymbol - [`KT-56543`](https://youtrack.jetbrains.com/issue/KT-56543) LL FIR: rework lazy transformers so transformers modify only declarations they suppose to - [`KT-56721`](https://youtrack.jetbrains.com/issue/KT-56721) K2: FirExtensionDeclarationsSymbolProvider: java.lang.IllegalStateException: Recursive update - [`KT-50253`](https://youtrack.jetbrains.com/issue/KT-50253) Analysis API: Solve issues with ProcessCancelledException - [`KT-56800`](https://youtrack.jetbrains.com/issue/KT-56800) K2 IDE: optimize deprecation calculation for symbols - [`KT-55006`](https://youtrack.jetbrains.com/issue/KT-55006) Analysis API does not transform Java type refs for callable symbol return types - [`KT-57256`](https://youtrack.jetbrains.com/issue/KT-57256) AA FIR: Reduce lazy resolve phase for deprecation status - [`KT-57619`](https://youtrack.jetbrains.com/issue/KT-57619) K2: CFG for class initializer is not correctly built in reversed resolve mode - [`KT-58141`](https://youtrack.jetbrains.com/issue/KT-58141) K2: AA FIR: impossible to restore symbol for declaration with annotation with argument inside type - [`KT-57462`](https://youtrack.jetbrains.com/issue/KT-57462) Symbol Light Classes: SymbolLightFieldForProperty should retrieve annotations not from KtPropertySymbol, but from the corresponding backing field - [`KT-54864`](https://youtrack.jetbrains.com/issue/KT-54864) Analysis API: add function to get expect KtSymbol list by actual KtSymbol - [`KT-56763`](https://youtrack.jetbrains.com/issue/KT-56763) Analysis API: `.KtSourceModuleImpl is missing in the map.` on symbol restore when symbol cannot be seen from the use-site module - [`KT-56617`](https://youtrack.jetbrains.com/issue/KT-56617) Analysis API: optimize KtFirSymbolProviderByJavaPsi.getNamedClassSymbol - [`KT-54430`](https://youtrack.jetbrains.com/issue/KT-54430) K2: .getAllOverriddenSymbols() returns invalid results ##### Backend. Native. Debug - [`KT-55440`](https://youtrack.jetbrains.com/issue/KT-55440) Kotlin/Native debugger: inline function parameters are not visible during debugging ##### Backend. Wasm - [`KT-58293`](https://youtrack.jetbrains.com/issue/KT-58293) Wasm: ReferenceError: e is not defined in kotlin.test.jsThrow - [`KT-58931`](https://youtrack.jetbrains.com/issue/KT-58931) Wasm tests are failing to start on Kotlin 1.9.0-Beta - [`KT-58188`](https://youtrack.jetbrains.com/issue/KT-58188) Restore binary compatibility of PlatformDiagnosticSuppressor.shouldReportUnusedParameter - [`KT-57136`](https://youtrack.jetbrains.com/issue/KT-57136) K/Wasm: Restrict non-external types in JS interop - [`KT-57060`](https://youtrack.jetbrains.com/issue/KT-57060) Clarify the lack of support for dynamic in Kotlin/Wasm - [`KT-56955`](https://youtrack.jetbrains.com/issue/KT-56955) K/Wasm: Support restricted version of K/JS `js(code)` - [`KT-57276`](https://youtrack.jetbrains.com/issue/KT-57276) Wasm: "Body not found for function" error when compiling konform library with Kotlin/Wasm support - [`KT-56976`](https://youtrack.jetbrains.com/issue/KT-56976) K/Wasm bug with calling override of external function with default parameters ##### Compiler ##### New Features - [`KT-55333`](https://youtrack.jetbrains.com/issue/KT-55333) Allow secondary constructors in value classes with bodies - [`KT-54944`](https://youtrack.jetbrains.com/issue/KT-54944) `@Volatile` support in native - [`KT-54746`](https://youtrack.jetbrains.com/issue/KT-54746) Deprecate with ERROR JvmDefault annotation and old -Xjvm-default modes - [`KT-47902`](https://youtrack.jetbrains.com/issue/KT-47902) Do not propagate method deprecation through overrides - [`KT-29378`](https://youtrack.jetbrains.com/issue/KT-29378) K2: rework warnings/errors for equality/identity operators on incompatible types - [`KT-57477`](https://youtrack.jetbrains.com/issue/KT-57477) False-positive overload resolution ambiguity in case of lambda without arguments - [`KT-57010`](https://youtrack.jetbrains.com/issue/KT-57010) Kotlin/Native: make it possible to compile bitcode in a separate compiler invocation - [`KT-55691`](https://youtrack.jetbrains.com/issue/KT-55691) K2: Avoid inferring Nothing? in presence of other constraints (beside type parameter bounds) - [`KT-46288`](https://youtrack.jetbrains.com/issue/KT-46288) Unexpected behavior of extension function on lambda with suspend receiver - [`KT-24779`](https://youtrack.jetbrains.com/issue/KT-24779) Inconsistent smart cast behavior for bound data flow values ##### Performance Improvements - [`KT-23397`](https://youtrack.jetbrains.com/issue/KT-23397) Optimize out field for property delegate when it's safe (JVM) - [`KT-56906`](https://youtrack.jetbrains.com/issue/KT-56906) FIR: Use cached instance of FirImplicitTypeRefImpl in FIR builders - [`KT-56276`](https://youtrack.jetbrains.com/issue/KT-56276) LanguageVersion.getVersionString() allocates 5k objects on project opening ##### Fixes - [`KT-57784`](https://youtrack.jetbrains.com/issue/KT-57784) "NullPointerException: Parameter specified as non-null is null:" with enum, companion object, 'entries' and map - [`KT-55217`](https://youtrack.jetbrains.com/issue/KT-55217) K2: support callable reference conversions on top-level expressions - [`KT-57232`](https://youtrack.jetbrains.com/issue/KT-57232) K2: build Space JVM (master) - [`KT-59079`](https://youtrack.jetbrains.com/issue/KT-59079) "AE: SyntheticAccessorLowering should not attempt to modify other files!" with callable reference to constructor with value class parameter - [`KT-58837`](https://youtrack.jetbrains.com/issue/KT-58837) Partial linkage fails to report any compiler message on Windows when launched through Gradle plugin - [`KT-57602`](https://youtrack.jetbrains.com/issue/KT-57602) K2: Rework member scope of types having projection arguments for covariant parameters - [`KT-55171`](https://youtrack.jetbrains.com/issue/KT-55171) Put new contracts syntax under a feature flag - [`KT-58719`](https://youtrack.jetbrains.com/issue/KT-58719) K2: false-positive INVISIBLE_REFERENCE error in case of importing an internal abstract class - [`KT-56030`](https://youtrack.jetbrains.com/issue/KT-56030) \[K2/N] Support Objective-C overloading by param names only - [`KT-57510`](https://youtrack.jetbrains.com/issue/KT-57510) K2: Data class equals/hashCode/toString methods are not written to Klib metadata - [`KT-56331`](https://youtrack.jetbrains.com/issue/KT-56331) K2: compiler backend crash on usage of expected function with default arguments - [`KT-53846`](https://youtrack.jetbrains.com/issue/KT-53846) K2 / Context receivers: ClassCastException on secondary constructor of class with context receiver - [`KT-58621`](https://youtrack.jetbrains.com/issue/KT-58621) K2: Private class shadows public function defined in the same package - [`KT-59102`](https://youtrack.jetbrains.com/issue/KT-59102) K2: constant evaluator does not provide Long type on shl - [`KT-59066`](https://youtrack.jetbrains.com/issue/KT-59066) \[K2] delegation leads to "IllegalStateException: Expected some types" - [`KT-56074`](https://youtrack.jetbrains.com/issue/KT-56074) K2: build Space JVM (snapshot 2022.3) - [`KT-58787`](https://youtrack.jetbrains.com/issue/KT-58787) KAPT: "NullPointerException: null cannot be cast to non-null type" with delegate - [`KT-57022`](https://youtrack.jetbrains.com/issue/KT-57022) K2 IllegalStateException in signature computation - [`KT-56792`](https://youtrack.jetbrains.com/issue/KT-56792) K2: build kotlinpoet - [`KT-57373`](https://youtrack.jetbrains.com/issue/KT-57373) K2: FIR properties synthesized when implementing interface by delegation don't have accessors - [`KT-56583`](https://youtrack.jetbrains.com/issue/KT-56583) K1: Implement opt-in for integer cinterop conversions - [`KT-59030`](https://youtrack.jetbrains.com/issue/KT-59030) \[PL] Workaround for broken `@Deprecated` annotations in c-interop KLIBs - [`KT-58618`](https://youtrack.jetbrains.com/issue/KT-58618) K2: Local property delegates cannot infer generic return type - [`KT-36770`](https://youtrack.jetbrains.com/issue/KT-36770) Prohibit unsafe calls with expected `@NotNull` T and given Kotlin generic parameter with nullable bound - [`KT-56739`](https://youtrack.jetbrains.com/issue/KT-56739) K2: build Space iOS - [`KT-57131`](https://youtrack.jetbrains.com/issue/KT-57131) K2: stdlib test compilation fails on ListTest.kt in FirJvmMangleComputer - [`KT-58137`](https://youtrack.jetbrains.com/issue/KT-58137) K2: ISE "Usage of default value argument for this annotation is not yet possible" when instantiating Kotlin annotation with default parameter from another module - [`KT-58897`](https://youtrack.jetbrains.com/issue/KT-58897) K2: False positive unresolved reference with same-named enum class and its entry - [`KT-40903`](https://youtrack.jetbrains.com/issue/KT-40903) Forbid actual member in expect class - [`KT-30905`](https://youtrack.jetbrains.com/issue/KT-30905) Expect var property with default public setter matches with actual var property with private setter - [`KT-56172`](https://youtrack.jetbrains.com/issue/KT-56172) K2: Fix reporting of PRIVATE_CLASS_MEMBER_FROM_INLINE error - [`KT-56171`](https://youtrack.jetbrains.com/issue/KT-56171) Implement deprecation warning for missing PRIVATE_CLASS_MEMBER_FROM_INLINE error - [`KT-27261`](https://youtrack.jetbrains.com/issue/KT-27261) Contracts for infix functions don't work (for receivers and parameters) - [`KT-56927`](https://youtrack.jetbrains.com/issue/KT-56927) Enum with secondary constructor can't be compiled with K2 using JS/Native backend - [`KT-53568`](https://youtrack.jetbrains.com/issue/KT-53568) Partial linkage: absent class as type parameter bound causes failure of `compileProductionExecutableKotlinJs` - [`KT-53608`](https://youtrack.jetbrains.com/issue/KT-53608) Partial linkage: Kotlin/JS fails with IllegalStateException: "Validation failed in file" when overridden declaration was visible, but now private - [`KT-53663`](https://youtrack.jetbrains.com/issue/KT-53663) Partial linkage: usage of property which becomes abstract: no IrLinkageError, but AssertionError in Native backend instead - [`KT-56013`](https://youtrack.jetbrains.com/issue/KT-56013) K2. a set of errors about local properties are missing - [`KT-53939`](https://youtrack.jetbrains.com/issue/KT-53939) Partial linkage: with turning object into class link*Native and js*Test tasks fail - [`KT-53938`](https://youtrack.jetbrains.com/issue/KT-53938) Partial linkage: with turning interface into class and using as second parent Native build fails - [`KT-53941`](https://youtrack.jetbrains.com/issue/KT-53941) Partial linkage: with turning class into object accessing member via parameterless constructor does not fail - [`KT-53970`](https://youtrack.jetbrains.com/issue/KT-53970) Partial linkage: on turning nested class into inner JS tasks are successful, Native build fails - [`KT-53971`](https://youtrack.jetbrains.com/issue/KT-53971) Partial linkage: turning inner class into nested: without usage in executable Native is successful, JavaScript fails - [`KT-53972`](https://youtrack.jetbrains.com/issue/KT-53972) Partial linkage: turning inner class into nested: with usage in executable Native fails with NPE in backend - [`KT-54045`](https://youtrack.jetbrains.com/issue/KT-54045) Partial linkage: turning class into type alias + calculating implicit function type: build fails with UninitializedPropertyAccessException: "lateinit property parent has not been initialized" - [`KT-54046`](https://youtrack.jetbrains.com/issue/KT-54046) Partial linkage: turning type alias into class + using it as type: build fails with AssertionError: "Expected exactly one delegating constructor call but none encountered" - [`KT-53887`](https://youtrack.jetbrains.com/issue/KT-53887) Partial linkage: turning from enum to regular class + reference to enum contant causes compileProductionExecutableKotlinJs fail with IllegalStateException - [`KT-54047`](https://youtrack.jetbrains.com/issue/KT-54047) Partial linkage: reference to removed enum const causes JS fail with "IllegalStateException: Validation failed in file" - [`KT-54048`](https://youtrack.jetbrains.com/issue/KT-54048) Partial linkage: reference to removed enum const in runtime causes Native fail with IllegalStateException at IrBindablePublicSymbolBase.getOwner() - [`KT-53995`](https://youtrack.jetbrains.com/issue/KT-53995) Partial linkage: on turning class to abstract and direct constructor call Naive fails, JavaScript is successful - [`KT-43527`](https://youtrack.jetbrains.com/issue/KT-43527) `@ExtensionFunctionType` is allowed on function types with no parameters and leads to backend exception - [`KT-55316`](https://youtrack.jetbrains.com/issue/KT-55316) K2. IllegalStateException on incorrect import directive name - [`KT-57570`](https://youtrack.jetbrains.com/issue/KT-57570) Remove source code excerpts from platform type nullability assertion exceptions - [`KT-56073`](https://youtrack.jetbrains.com/issue/KT-56073) K2: build Exposed - [`KT-47932`](https://youtrack.jetbrains.com/issue/KT-47932) Report errors on cycles in annotation parameter types - [`KT-38871`](https://youtrack.jetbrains.com/issue/KT-38871) Kotlin Gradle DSL, MPP: UNUSED_VARIABLE when configuring a sourceset with delegated property - [`KT-46344`](https://youtrack.jetbrains.com/issue/KT-46344) No error for a super class constructor call on a function interface in supertypes list - [`KT-56609`](https://youtrack.jetbrains.com/issue/KT-56609) K2: False positive NULL_FOR_NONNULL_TYPE with -Xjsr305=strict and `@Nullable` annotation Java parameter - [`KT-56656`](https://youtrack.jetbrains.com/issue/KT-56656) K1/K2: inconsistent NOTHING_TO_OVERRIDE with complex nullable annotations - [`KT-58332`](https://youtrack.jetbrains.com/issue/KT-58332) K2: local fun with suspend type is not marked as suspend in IR - [`KT-57991`](https://youtrack.jetbrains.com/issue/KT-57991) K2: Modifier 'suspend' is not applicable to 'anonymous function' - [`KT-54294`](https://youtrack.jetbrains.com/issue/KT-54294) K2: "Not all type variables found" in builder inference with type parameters inferred through a union of two branches - [`KT-58564`](https://youtrack.jetbrains.com/issue/KT-58564) \[PL] Annotations with unlinked parameters are not removed - [`KT-52597`](https://youtrack.jetbrains.com/issue/KT-52597) Provide Alpha Support for Multiplatform in the K2 platform - [`KT-58523`](https://youtrack.jetbrains.com/issue/KT-58523) K2: reference is resolved to imported type-alias instead of identically named top-level property - [`KT-57098`](https://youtrack.jetbrains.com/issue/KT-57098) Native: avoid object initialization while accessing const val - [`KT-57973`](https://youtrack.jetbrains.com/issue/KT-57973) 32-th default value in inline classes override function is not used - [`KT-57714`](https://youtrack.jetbrains.com/issue/KT-57714) "IllegalStateException: \Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
â™» Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.
This PR has been generated by Mend Renovate. View repository job log here.