Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prepare 1.21.0-RC1 #4900

Merged
merged 8 commits into from
Jun 2, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
2 changes: 1 addition & 1 deletion build-logic/src/main/kotlin/Versions.kt
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
object Versions {

const val DETEKT: String = "1.20.0"
const val DETEKT: String = "1.21.0-RC1"
const val SNAPSHOT_NAME: String = "main"
const val JVM_TARGET: String = "1.8"

Expand Down
5 changes: 0 additions & 5 deletions config/detekt/detekt.yml
Original file line number Diff line number Diff line change
Expand Up @@ -142,11 +142,6 @@ potential-bugs:
active: true
MapGetWithNotNullAssertionOperator:
active: true
MissingPackageDeclaration:
active: true
excludes:
- '**/*.kts'
- '**/build-logic/**'
UnconditionalJumpStatementInLoop:
active: true
UnreachableCatchBlock:
Expand Down
2 changes: 1 addition & 1 deletion detekt-core/src/main/resources/default-detekt-config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -452,7 +452,7 @@ potential-bugs:
MapGetWithNotNullAssertionOperator:
active: true
MissingPackageDeclaration:
active: true
active: false
excludes: ['**/*.kts']
MissingWhenCase:
active: true
Expand Down
29 changes: 29 additions & 0 deletions detekt-gradle-plugin/build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -177,4 +177,33 @@ if (signingKey.isNullOrBlank() || signingPwd.isNullOrBlank()) {
}
}

afterEvaluate {
publishing {
publications.filterIsInstance<MavenPublication>().forEach {
it.pom {
description.set("The official Detekt Gradle Plugin")
name.set("detekt-gradle-plugin")
url.set("https://detekt.dev")
licenses {
license {
name.set("The Apache Software License, Version 2.0")
url.set("https://www.apache.org/licenses/LICENSE-2.0.txt")
distribution.set("repo")
}
}
developers {
developer {
id.set("Detekt Developers")
name.set("Detekt Developers")
email.set("info@detekt.dev")
}
}
scm {
url.set("https://github.com/detekt/detekt")
}
}
}
}
}

val String.byProperty: String? get() = findProperty(this) as? String
Original file line number Diff line number Diff line change
Expand Up @@ -553,9 +553,7 @@ private val SAMPLE_ACTIVITY_LAYOUT = """
/>
""".trimIndent()

private val SAMPLE_ACTIVITY_USING_VIEW_BINDING = """
package io.gitlab.arturbosch.detekt.app

private val SAMPLE_ACTIVITY_USING_VIEW_BINDING = """
import android.app.Activity
import android.os.Bundle
import android.view.LayoutInflater
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,14 +7,12 @@ import io.gitlab.arturbosch.detekt.api.Entity
import io.gitlab.arturbosch.detekt.api.Issue
import io.gitlab.arturbosch.detekt.api.Rule
import io.gitlab.arturbosch.detekt.api.Severity
import io.gitlab.arturbosch.detekt.api.internal.ActiveByDefault
import org.jetbrains.kotlin.psi.KtFile
import org.jetbrains.kotlin.psi.KtPackageDirective

/**
* Reports when the package declaration is missing.
*/
@ActiveByDefault(since = "1.21.0")
class MissingPackageDeclaration(config: Config = Config.empty) : Rule(config) {

override val issue = Issue(
Expand Down
3 changes: 2 additions & 1 deletion scripts/release.sh
Original file line number Diff line number Diff line change
@@ -1,7 +1,8 @@
#!/usr/bin/env sh
gradle publishToMavenLocal || exit
gradle build || exit
gradle publishAllToMavenCentral --max-workers 1 || exit
gradle publishPlugins || exit
gradle :detekt-gradle-plugin:publishPlugins || exit
gradle githubRelease || exit
gradle applyDocVersion || exit
gradle closeAndReleaseRepository || exit
91 changes: 91 additions & 0 deletions website/docs/introduction/changelog 1.x.x.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,97 @@ keywords: [changelog, release-notes, migration]
sidebar_position: 1
---

#### 1.21.0-RC1 - 2022-06-02

##### Notable Changes

- We enabled ~30 new rules by default which we believe are now stable enough. - [#4875](https://github.com/detekt/detekt/pull/4875)
- We added **3** new Rules to Detekt
- `NullableBooleanCheck` - [#4872](https://github.com/detekt/detekt/pull/4872)
- `CouldBeSequence` - [#4855](https://github.com/detekt/detekt/pull/4855)
- `UnnecessaryBackticks` - [#4764](https://github.com/detekt/detekt/pull/4764)
- We now allow users and rule authors to specify a **reason** for every value in the config file - [#4611](https://github.com/detekt/detekt/pull/4611)
- We now report as warnings the in the config file that should converted to be an array - [#4793](https://github.com/detekt/detekt/pull/4793)
- We added a dependency on **ConTester** to help us verify concurrency scenarios for Detekt - [#4672](https://github.com/detekt/detekt/pull/4672)
- For contributors: we restructured our build setup to be use **Gradle composite build** - [#4751](https://github.com/detekt/detekt/pull/4751)

##### Changelog

- Simplify regular expressions - [#4893](https://github.com/detekt/detekt/pull/4893)
- Remove redundant character escape in RegExp - [#4892](https://github.com/detekt/detekt/pull/4892)
- Reformat Markdown files to comply with the spec - [#4891](https://github.com/detekt/detekt/pull/4891)
- UnnecessaryInnerClass: fix false negative with `this` references - [#4884](https://github.com/detekt/detekt/pull/4884)
- UselessCallOnNotNull: fix false positive for unresolved types - [#4880](https://github.com/detekt/detekt/pull/4880)
- Update MagicNumber rule to exclude .kts files - [#4877](https://github.com/detekt/detekt/pull/4877)
- CanBeNonNullable: fix false positives for parameterized types - [#4870](https://github.com/detekt/detekt/pull/4870)
- UnnecessaryInnerClass: fix false positives labeled expression to outer class - [#4865](https://github.com/detekt/detekt/pull/4865)
- UnnecessaryInnerClass: add test for safe qualified expressions - [#4864](https://github.com/detekt/detekt/pull/4864)
- Fix a confusing Regex in the Compose webpage - [#4852](https://github.com/detekt/detekt/pull/4852)
- Fix edit URLs for the website - [#4850](https://github.com/detekt/detekt/pull/4850)
- detektGenerateConfig adds the configuration of plugins - [#4844](https://github.com/detekt/detekt/pull/4844)
- Update dependency prism-react-renderer to v1.3.3 - [#4833](https://github.com/detekt/detekt/pull/4833)
- Search in all versions.properties, not just the first one #4830 - [#4831](https://github.com/detekt/detekt/pull/4831)
- Improve exception message - [#4823](https://github.com/detekt/detekt/pull/4823)
- Fix ValShouldBeVar false positive inside unknown type - [#4820](https://github.com/detekt/detekt/pull/4820)
- Add a recent conference talk link - [#4819](https://github.com/detekt/detekt/pull/4819)
- False positive for unused imports #4815 - [#4818](https://github.com/detekt/detekt/pull/4818)
- Revert "Display dynamic --jvm-target values when using --help flag (#4694)" - [#4816](https://github.com/detekt/detekt/pull/4816)
- UnnecessaryAbstractClass: report only the class name - [#4808](https://github.com/detekt/detekt/pull/4808)
- Fix wrong replacement suggestion for UnnecessaryFilter - [#4807](https://github.com/detekt/detekt/pull/4807)
- UseOrEmpty: fix false positive for indexing operator calls with type parameters - [#4804](https://github.com/detekt/detekt/pull/4804)
- ExplicitCollectionElementAccessMethod: fix false positive for get operators with type parameters - [#4803](https://github.com/detekt/detekt/pull/4803)
- Add tests for #4786 - [#4801](https://github.com/detekt/detekt/pull/4801)
- Add documentation link for rules in html report - [#4799](https://github.com/detekt/detekt/pull/4799)
- Improve rule documentaion and smell message of NamedArguments - [#4796](https://github.com/detekt/detekt/pull/4796)
- Improve issue description and smell message of DestructuringDeclarationWithTooManyEntries - [#4795](https://github.com/detekt/detekt/pull/4795)
- NestedScopeFunctions - Add rule for nested scope functions - [#4788](https://github.com/detekt/detekt/pull/4788)
- Partially drop redundant usage of "dry run" in Gradle plugin tests - [#4776](https://github.com/detekt/detekt/pull/4776)
- Allow additionalJavaSourceRootPaths to be defined on @KotlinCoreEnvironmentTest - [#4771](https://github.com/detekt/detekt/pull/4771)
- Report KDoc comments that refer to non-public properties of a class - [#4768](https://github.com/detekt/detekt/pull/4768)
- Self-inspect the detekt-gradle-plugin - [#4765](https://github.com/detekt/detekt/pull/4765)
- Pass args to DetektInvoker as List&lt;String&gt; - [#4762](https://github.com/detekt/detekt/pull/4762)
- Cleanup Gradle Plugin Publications - [#4752](https://github.com/detekt/detekt/pull/4752)
- Break a dependency between `detekt-gradle-plugin` and `detekt-utils` - [#4748](https://github.com/detekt/detekt/pull/4748)
- Remove suspend lambda rule with CoroutineScope receiver due to not de… - [#4747](https://github.com/detekt/detekt/pull/4747)
- `VarCouldBeVal`: Add configuration flag `ignoreLateinitVar` - [#4745](https://github.com/detekt/detekt/pull/4745)
- UnnecessaryInnerClass: fix false positive with references to function type variables - [#4738](https://github.com/detekt/detekt/pull/4738)
- Fix false positive on VarCouldBeVal in generic classes - [#4733](https://github.com/detekt/detekt/pull/4733)
- OutdatedDocumentation: fix false positive with no primary constructor - [#4728](https://github.com/detekt/detekt/pull/4728)
- Android Gradle: add javac intermediates to classpath - [#4723](https://github.com/detekt/detekt/pull/4723)
- OptionalWhenBraces: fix false negative when the single statement has comments inside - [#4722](https://github.com/detekt/detekt/pull/4722)
- Document pre-commit hook for staged files - [#4711](https://github.com/detekt/detekt/pull/4711)
- Enable rules by default for `1.21` - [#4643](https://github.com/detekt/detekt/issues/4643)

##### Dependency Updates

- Update dependency org.jetbrains.kotlinx:kotlinx-coroutines-core to v1.6.2 - [#4868](https://github.com/detekt/detekt/pull/4868)
- Update dependency com.android.tools.build:gradle to v7.2.1 - [#4861](https://github.com/detekt/detekt/pull/4861)
- Update plugin binaryCompatibilityValidator to v0.10.0 - [#4837](https://github.com/detekt/detekt/pull/4837)
- Update dependency io.mockk:mockk to v1.12.4 - [#4829](https://github.com/detekt/detekt/pull/4829)
- Update dependency com.android.tools.build:gradle to v7.2.0 - [#4824](https://github.com/detekt/detekt/pull/4824)
- Add dependency-analysis plugin and implement some recommendations - [#4798](https://github.com/detekt/detekt/pull/4798)
- Add dependency on slf4j-nop to silence warning - [#4775](https://github.com/detekt/detekt/pull/4775)
- Update plugin dokka to v1.6.21 - [#4770](https://github.com/detekt/detekt/pull/4770)
- Update org.jetbrains.kotlin to v1.6.21 - [#4737](https://github.com/detekt/detekt/pull/4737)
- Update dependency com.github.breadmoirai:github-release to v2.3.7 - [#4734](https://github.com/detekt/detekt/pull/4734)
- Update plugin binaryCompatibilityValidator to v0.9.0 - [#4729](https://github.com/detekt/detekt/pull/4729)

##### Housekeeping & Refactorings

- Declare nested test classes as non-static - [#4894](https://github.com/detekt/detekt/pull/4894)
- Remove deprecated usages in gradle-plugin test - [#4889](https://github.com/detekt/detekt/pull/4889)
- Remove reference to contributor list - [#4871](https://github.com/detekt/detekt/pull/4871)
- Add missing image - [#4834](https://github.com/detekt/detekt/pull/4834)
- Upgrade to GE enterprise 3.10 - [#4802](https://github.com/detekt/detekt/pull/4802)
- Fix broken snapshot publishing - [#4783](https://github.com/detekt/detekt/pull/4783)
- Remove pending Gradle version milestones from comments - [#4777](https://github.com/detekt/detekt/pull/4777)
- Add more tests for Annotation Suppressor - [#4774](https://github.com/detekt/detekt/pull/4774)
- fix: add test case that fails if environment is not properly set up - [#4769](https://github.com/detekt/detekt/pull/4769)
- Disable UnusedImports for the Detekt project - [#4741](https://github.com/detekt/detekt/pull/4741)
- Remove Unnecesary @Nested - [#4740](https://github.com/detekt/detekt/pull/4740)
- Update the argsfile to unblock `runWithArgsFile` failing locally - [#4718](https://github.com/detekt/detekt/pull/4718)

See all issues at: [1.21.0](https://github.com/detekt/detekt/milestone/86)

#### 1.20.0 - 2022-04-14

Expand Down
2 changes: 1 addition & 1 deletion website/docusaurus.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -179,7 +179,7 @@ const config = {
}),

customFields: {
detektVersion: '1.20.0'
detektVersion: '1.21.0-RC1'
},
};

Expand Down