activate unused rewrite checks failOnDryRunResults

Vincent Potucek · Vincent Potucek · commit a99b0df24e80 · 2025-07-01T16:21:39.000+02:00
diff --git a/gradle/plugins/common/src/main/kotlin/junitbuild.java-library-conventions.gradle.kts b/gradle/plugins/common/src/main/kotlin/junitbuild.java-library-conventions.gradle.kts
@@ -18,11 +18,15 @@ plugins {
 
 rewrite {
 	activeRecipe("org.openrewrite.java.migrate.UpgradeToJava17")
+	activeRecipe("org.openrewrite.staticanalysis.RemoveUnusedPrivateMethods")
+	activeRecipe("org.openrewrite.staticanalysis.MissingOverrideAnnotation")
+	failOnDryRunResults = true
 }
 
 dependencies {
 	rewrite(platform("org.openrewrite.recipe:rewrite-recipe-bom:latest.release"))
 	rewrite("org.openrewrite.recipe:rewrite-migrate-java")
+	rewrite("org.openrewrite.recipe:rewrite-static-analysis")
 }
 
 val mavenizedProjects: List<Project> by rootProject.extra
@@ -277,6 +281,10 @@ tasks {
 	checkstyleMain {
 		config = resources.text.fromFile(checkstyle.configDirectory.file("checkstyleMain.xml"))
 	}
+	assemble {
+		dependsOn("rewriteDryRun")
+		mustRunAfter("rewriteDryRun")
+	}
 	checkstyleTest {
 		config = resources.text.fromFile(checkstyle.configDirectory.file("checkstyleTest.xml"))
 	}

Original file line number	Diff line number	Diff line change
`@@ -18,11 +18,15 @@ plugins {`
`18`	`18`
`19`	`19`	`rewrite {`
`20`	`20`	`activeRecipe("org.openrewrite.java.migrate.UpgradeToJava17")`
	`21`	`+ activeRecipe("org.openrewrite.staticanalysis.RemoveUnusedPrivateMethods")`
	`22`	`+ activeRecipe("org.openrewrite.staticanalysis.MissingOverrideAnnotation")`
	`23`	`+ failOnDryRunResults = true`
`21`	`24`	`}`
`22`	`25`
`23`	`26`	`dependencies {`
`24`	`27`	`rewrite(platform("org.openrewrite.recipe:rewrite-recipe-bom:latest.release"))`
`25`	`28`	`rewrite("org.openrewrite.recipe:rewrite-migrate-java")`
	`29`	`+ rewrite("org.openrewrite.recipe:rewrite-static-analysis")`
`26`	`30`	`}`
`27`	`31`
`28`	`32`	`val mavenizedProjects: List<Project> by rootProject.extra`
`@@ -277,6 +281,10 @@ tasks {`
`277`	`281`	`checkstyleMain {`
`278`	`282`	`config = resources.text.fromFile(checkstyle.configDirectory.file("checkstyleMain.xml"))`
`279`	`283`	`}`
	`284`	`+ assemble {`
	`285`	`+ dependsOn("rewriteDryRun")`
	`286`	`+ mustRunAfter("rewriteDryRun")`
	`287`	`+ }`
`280`	`288`	`checkstyleTest {`
`281`	`289`	`config = resources.text.fromFile(checkstyle.configDirectory.file("checkstyleTest.xml"))`
`282`	`290`	`}`