Add error-prone.picnic.tech

Author: Vincent Potucek

build.gradle

@@ -12,6 +12,7 @@ repositories {
 apply from: rootProject.file('gradle/java-publish.gradle')
 apply from: rootProject.file('gradle/changelog.gradle')
 allprojects {
+	apply from: rootProject.file('gradle/error-prone.gradle')
 	apply from: rootProject.file('gradle/rewrite.gradle')
 	apply from: rootProject.file('gradle/spotless.gradle')
 }

gradle/error-prone.gradle

@@ -0,0 +1,66 @@
+import static java.lang.System.getenv
+
+apply plugin: 'net.ltgt.errorprone'
+
+dependencies {
+	errorprone('com.google.errorprone:error_prone_core:2.42.0')
+	errorprone('tech.picnic.error-prone-support:error-prone-contrib:0.25.0')
+	errorprone('tech.picnic.error-prone-support:refaster-runner:0.25.0')
+	constraints {
+		errorprone('com.google.guava:guava') {
+			version {
+				require('33.4.8-jre')
+			}
+			because('Older versions use deprecated methods in sun.misc.Unsafe')
+			// https://github.com/junit-team/junit-framework/pull/5039#discussion_r2414490581
+		}
+	}
+}
+
+tasks.withType(JavaCompile).configureEach {
+	options.errorprone {
+		errorproneArgs.add('-XepOpt:Refaster:NamePattern=^(?!.*Rules\\$).*')
+		error(
+				'DirectReturn',
+				// 'ConstantNaming',
+				// 'LexicographicalAnnotationAttributeListing',
+				// 'LexicographicalAnnotationListing',
+				// 'MissingOverride',
+				// 'NonStaticImport',
+				// 'OptionalMapUnusedValue',
+				// 'OptionalOfRedundantMethod',
+				// 'RedundantSetterCall',
+				// 'RedundantStringConversion',
+				// 'RedundantStringEscape',
+				// 'StaticImport',
+				// 'StringJoin',
+				// 'UnnecessaryCheckNotNull',
+				// 'UnnecessaryTypeArgument',
+				// 'UnusedAnonymousClass',
+				// 'UnusedCollectionModifiedInPlace',
+				)
+		if (!getenv().containsKey('CI') && getenv('IN_PLACE')?.toBoolean()) {
+			errorproneArgs.addAll(
+					'-XepPatchLocation:IN_PLACE',
+					'-XepPatchChecks:' +
+					'DirectReturn,'
+					// 'ConstantNaming,' +
+					// 'MissingOverride,' +
+					// 'StaticImport,' +
+					// LexicographicalAnnotationAttributeListing,' +
+					// LexicographicalAnnotationListing,' +
+					// NonStaticImport,' +
+					// OptionalMapUnusedValue,' +
+					// OptionalOfRedundantMethod,' +
+					// RedundantSetterCall,' +
+					// RedundantStringConversion,' +
+					// RedundantStringEscape,' +
+					// StringJoin,' +
+					// UnnecessaryCheckNotNull,' +
+					// UnnecessaryTypeArgument,' +
+					// UnusedAnonymousClass,' +
+					// UnusedCollectionModifiedInPlace,'
+					)
+		}
+	}
+}

rewrite.yml

@@ -10,7 +10,6 @@ tags:
   - cleanup
 recipeList:
   - org.openrewrite.gradle.EnableGradleBuildCache
-  - org.openrewrite.gradle.EnableGradleParallelExecution
   - org.openrewrite.gradle.GradleBestPractices
   - org.openrewrite.java.RemoveUnusedImports
   - org.openrewrite.java.format.NormalizeFormat

settings.gradle

@@ -23,6 +23,7 @@ plugins {
 	id 'com.gradle.develocity' version '4.2.1'
 	// https://github.com/equodev/equo-ide/blob/main/plugin-gradle/CHANGELOG.md
 	id 'dev.equo.ide' version '1.7.8' apply false
+	id 'net.ltgt.errorprone' version '4.3.0' apply false
 	id 'org.openrewrite.rewrite' version '7.17.0' apply false
 }