From d7617251f49127891c60e567d01a67d3fb955f10 Mon Sep 17 00:00:00 2001 From: Nicola Corti Date: Thu, 18 Nov 2021 16:36:37 +0000 Subject: [PATCH] Prepare Detekt 1.19.0-RC2 (#4293) --- build-logic/src/main/kotlin/Versions.kt | 2 +- docs/_config.yml | 2 +- docs/pages/changelog 1.x.x.md | 44 +++++++++++++++++++++++-- 3 files changed, 43 insertions(+), 5 deletions(-) diff --git a/build-logic/src/main/kotlin/Versions.kt b/build-logic/src/main/kotlin/Versions.kt index 7d34990b4a0..0c446f19e42 100644 --- a/build-logic/src/main/kotlin/Versions.kt +++ b/build-logic/src/main/kotlin/Versions.kt @@ -1,6 +1,6 @@ object Versions { - const val DETEKT: String = "1.19.0-RC1" + const val DETEKT: String = "1.19.0-RC2" const val SNAPSHOT_NAME: String = "main" const val JVM_TARGET: String = "1.8" diff --git a/docs/_config.yml b/docs/_config.yml index 73b152de69e..1fe29f4d09e 100644 --- a/docs/_config.yml +++ b/docs/_config.yml @@ -117,4 +117,4 @@ description: "Meet detekt, a static code analysis tool for Kotlin." url: https://detekt.github.io baseurl: /detekt -detekt_version: 1.19.0-RC1 +detekt_version: 1.19.0-RC2 diff --git a/docs/pages/changelog 1.x.x.md b/docs/pages/changelog 1.x.x.md index f79d196fee3..17d90d82ad9 100644 --- a/docs/pages/changelog 1.x.x.md +++ b/docs/pages/changelog 1.x.x.md @@ -10,15 +10,18 @@ toc: true ##### Notable Changes -- We now offer an `ignoreAnnotated` configuration key that you can use on all your rules to suppress findings if inside an annotated block (e.g. `@Composable`) - [#4102](https://github.com/detekt/detekt/pull/4102) +- We now offer an `ignoreAnnotated` configuration key that you can use on all your rules to suppress findings if inside an annotated block (e.g. `@Composable`) - [#4102](https://github.com/detekt/detekt/pull/4102) and [#4241](https://github.com/detekt/detekt/pull/4241) - Report configuration is changing in the Gradle plugin. The `reports` extension on the `detekt` extension has been deprecated. See the Migration section below for steps to migrate to the new recommended configuration - [#3687](https://github.com/detekt/detekt/pull/3687) - The `ExplicitCollectionElementAccessMethod` rule is now a type-resolution only rule - [#4201](https://github.com/detekt/detekt/pull/4201) - The `InvalidPackageDeclaration` rule has been split to create the `MissingPackageDeclaration` rule - [#4149](https://github.com/detekt/detekt/pull/4149) - The `ForbiddenComment` rule now offers a `customMessage` configuration key - [#4126](https://github.com/detekt/detekt/pull/4126) - We bumped ktlint and updated the default enabled rules to mirror what ktlint is doing - [#4179](https://github.com/detekt/detekt/pull/4179) -- Add a new ConsoleReport format - [#4027](https://github.com/detekt/detekt/pull/4027) -- Gradle: We removed the `afterEvaluate` wrapper from the Android and KMM plugin - [#4159](https://github.com/detekt/detekt/pull/4159) +- Added a new LambdaParameterNaming rule, to enfornce a naming convention of paramter inside lambdas - [#4147](https://github.com/detekt/detekt/pull/4147) +- Added a new InjectDispatcher rule, to check if dispatchers are injectable - [#4222](https://github.com/detekt/detekt/pull/4222) +- Added a new ConsoleReport format - [#4027](https://github.com/detekt/detekt/pull/4027) +- Gradle: We added the `--auto-correct` cmdline option to gradle tasks - [#4202](https://github.com/detekt/detekt/pull/4202) +- Gradle: We removed the `afterEvaluate` wrapper from the Android and KMM plugin - [#4159](https://github.com/detekt/detekt/pull/4159) and [#4271](https://github.com/detekt/detekt/pull/4271) - We now test against Java 17 and stopped testing against Java 16 - [#4136](https://github.com/detekt/detekt/pull/4136) - Remove library specific configurations like Jetpack Compose and Dagger from the default config - [#4101](https://github.com/detekt/detekt/pull/4101) - Remove detekt-bom module - [#4043](https://github.com/detekt/detekt/pull/4043) @@ -77,6 +80,21 @@ tasks.detektMain { ##### Changelog +- Add documentation about how to configure Baseline task with type resolution - [#4285](https://github.com/detekt/detekt/pull/4285) +- Remove kotlin-gradle-plugin-api from runtime classpath - [#4275](https://github.com/detekt/detekt/pull/4275) +- Use appropriate annotations on source properties in Gradle tasks - [#4264](https://github.com/detekt/detekt/pull/4264) +- Replace usage of deprecated ConfigureUtil - [#4263](https://github.com/detekt/detekt/pull/4263) +- Fix test failure of ReportMergeSpec - [#4262](https://github.com/detekt/detekt/pull/4262) +- Revert "Remove afterEvaluate wrapper (#4159)" - [#4259](https://github.com/detekt/detekt/pull/4259) +- ExplicitCollectionElementAccessMethodSpec: does not report methods that is called on implicit receiver - [#4256](https://github.com/detekt/detekt/pull/4256) +- UnusedPrivateMember: fix false positive with operator `in` - [#4249](https://github.com/detekt/detekt/pull/4249) +- Introduce UseAnyOrNoneInsteadOfFind rule - [#4247](https://github.com/detekt/detekt/pull/4247) +- OptionalWhenBraces: fix false negative for nested when - [#4246](https://github.com/detekt/detekt/pull/4246) +- Handle MultiRules in Suppressors - [#4239](https://github.com/detekt/detekt/pull/4239) +- Fix UselessCallOnNotNull rule - [#4237](https://github.com/detekt/detekt/pull/4237) +- Make detekt a bit less noisy when mixing java and kotlin files - [#4231](https://github.com/detekt/detekt/pull/4231) +- Workaround for JDK 8 instability when reading config - [#4225](https://github.com/detekt/detekt/pull/4225) +- Define FunctionSignature - [#4176](https://github.com/detekt/detekt/pull/4176) - ForbiddenMethodCall: report overriding method calls - [#4205](https://github.com/detekt/detekt/pull/4205) - ObjectLiteralToLambda: fix false positive when using Java interfaces with default methods - [#4203](https://github.com/detekt/detekt/pull/4203) - Unit tests for TooGenericExceptionThrown - [#4198](https://github.com/detekt/detekt/pull/4198) @@ -107,12 +125,32 @@ tasks.detektMain { ##### Dependency Updates +- Gradle Publishing Plugin 0.17.0 - [#4270](https://github.com/detekt/detekt/pull/4270) +- Shadow 7.1.0 - [#4269](https://github.com/detekt/detekt/pull/4269) +- Dokka 1.5.31 - [#4268](https://github.com/detekt/detekt/pull/4268) +- Binary Compatibility Validator 0.8.0 - [#4267](https://github.com/detekt/detekt/pull/4267) +- Reflections 0.10.2 - [#4266](https://github.com/detekt/detekt/pull/4266) +- Upgrade to Gradle 7.3 - [#4254](https://github.com/detekt/detekt/pull/4254) - Dokka 1.5.30 - [#4114](https://github.com/detekt/detekt/pull/4114) - Kotlin 1.5.31 - [#4113](https://github.com/detekt/detekt/pull/4113) - Update dependencies - [#4065](https://github.com/detekt/detekt/pull/4065) ##### Housekeeping & Refactorings +- Minor typo fix and code refactoring - [#4284](https://github.com/detekt/detekt/pull/4284) +- Improve Tests of UnnecesaryLet - [#4282](https://github.com/detekt/detekt/pull/4282) +- Fix typo in UnnecessaryLet - [#4281](https://github.com/detekt/detekt/pull/4281) +- Fix typo in Gradle lib definition - [#4255](https://github.com/detekt/detekt/pull/4255) +- Rename DoubleMutabilityInCollectionSpec to DoubleMutabilityForCollectionSpec - [#4251](https://github.com/detekt/detekt/pull/4251) +- Simplify conditional checks to improve coverage - [#4221](https://github.com/detekt/detekt/pull/4221) +- Refactor NoTabs to remove DetektVisitor - [#4220](https://github.com/detekt/detekt/pull/4220) +- Fix typos and grammar in rule descriptions - [#4219](https://github.com/detekt/detekt/pull/4219) +- Use Kotlin's ArrayDeque implementation - [#4218](https://github.com/detekt/detekt/pull/4218) +- Update Kotlin docs URL - [#4217](https://github.com/detekt/detekt/pull/4217) +- Report UntilInsteadOfRangeTo for 'rangeTo' calls - [#4212](https://github.com/detekt/detekt/pull/4212) +- Add tests for merging reports - [#4199](https://github.com/detekt/detekt/pull/4199) +- Setup Gradle functional tests - [#4074](https://github.com/detekt/detekt/pull/4074) +- GitHub Actions cache fixes - [#3723](https://github.com/detekt/detekt/pull/3723) - Simplify where casts used unnecessarily - [#4213](https://github.com/detekt/detekt/pull/4213) - Don't specify Gradle Enterprise Gradle Plugin version - [#4210](https://github.com/detekt/detekt/pull/4210) - Fix baserule import in tests - [#4189](https://github.com/detekt/detekt/pull/4189)