Introduce error-prone-experimental module

PicnicSupermarket · Feb 4, 2024 · ba721eb · ba721eb
1 parent 947914a
commit ba721eb
Show file tree

Hide file tree

Showing 6 changed files with 110 additions and 2 deletions.
diff --git a/...prone-contrib/src/main/java/tech/picnic/errorprone/bugpatterns/IsInstanceLambdaUsage.java b/...prone-contrib/src/main/java/tech/picnic/errorprone/bugpatterns/IsInstanceLambdaUsage.java
@@ -24,7 +24,7 @@
  * A {@link BugChecker} that flags lambda expressions that can be replaced with a method reference
  * of the form {@code T.class::isInstance}.
  *
- * @see MethodReferenceUsage
+ * <p>See the `MethodReferenceUsage` check in the `error-prone-experimental` module.
  */
 // XXX: Consider folding this logic into the `MethodReferenceUsage` check.
 @AutoService(BugChecker.class)

diff --git a/error-prone-experimental/README.md b/error-prone-experimental/README.md
@@ -0,0 +1,6 @@
+# Experimental Error Prone checks
+
+This module contains Error Prone checks that are currently under development or
+evaluation. These checks may be works-in-progress or have uncertain impacts on
+code. Having this module allows for controlled experimentation and refinement
+before integration into production environments.
diff --git a/error-prone-experimental/pom.xml b/error-prone-experimental/pom.xml
@@ -0,0 +1,91 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 https://maven.apache.org/xsd/maven-4.0.0.xsd">
+    <modelVersion>4.0.0</modelVersion>
+
+    <parent>
+        <groupId>tech.picnic.error-prone-support</groupId>
+        <artifactId>error-prone-support</artifactId>
+        <version>0.14.1-SNAPSHOT</version>
+    </parent>
+
+    <artifactId>error-prone-experimental</artifactId>
+
+    <name>Picnic :: Error Prone Support :: Experimental</name>
+    <description>Experimental Error Prone checks.</description>
+    <url>https://error-prone.picnic.tech</url>
+
+    <dependencies>
+        <dependency>
+            <groupId>${groupId.error-prone}</groupId>
+            <artifactId>error_prone_annotation</artifactId>
+            <scope>provided</scope>
+        </dependency>
+        <dependency>
+            <groupId>${groupId.error-prone}</groupId>
+            <artifactId>error_prone_annotations</artifactId>
+            <scope>provided</scope>
+        </dependency>
+        <dependency>
+            <groupId>${groupId.error-prone}</groupId>
+            <artifactId>error_prone_check_api</artifactId>
+            <scope>provided</scope>
+        </dependency>
+        <dependency>
+            <groupId>${groupId.error-prone}</groupId>
+            <artifactId>error_prone_core</artifactId>
+            <scope>provided</scope>
+        </dependency>
+        <dependency>
+            <groupId>${groupId.error-prone}</groupId>
+            <artifactId>error_prone_test_helpers</artifactId>
+            <scope>provided</scope>
+        </dependency>
+        <dependency>
+            <groupId>${project.groupId}</groupId>
+            <artifactId>error-prone-utils</artifactId>
+            <scope>provided</scope>
+        </dependency>
+        <dependency>
+            <groupId>com.google.auto.service</groupId>
+            <artifactId>auto-service-annotations</artifactId>
+            <scope>provided</scope>
+        </dependency>
+        <dependency>
+            <groupId>com.google.guava</groupId>
+            <artifactId>guava</artifactId>
+            <scope>provided</scope>
+        </dependency>
+        <dependency>
+            <groupId>io.projectreactor</groupId>
+            <artifactId>reactor-core</artifactId>
+            <scope>test</scope>
+        </dependency>
+        <dependency>
+            <groupId>org.assertj</groupId>
+            <artifactId>assertj-core</artifactId>
+            <scope>test</scope>
+        </dependency>
+        <dependency>
+            <groupId>org.jspecify</groupId>
+            <artifactId>jspecify</artifactId>
+            <scope>provided</scope>
+        </dependency>
+        <dependency>
+            <groupId>org.junit.jupiter</groupId>
+            <artifactId>junit-jupiter-api</artifactId>
+            <scope>provided</scope>
+        </dependency>
+        <!-- XXX: Explicitly declared as a workaround for
+        https://github.com/pitest/pitest-junit5-plugin/issues/105. -->
+        <dependency>
+            <groupId>org.junit.jupiter</groupId>
+            <artifactId>junit-jupiter-engine</artifactId>
+            <scope>test</scope>
+        </dependency>
+        <dependency>
+            <groupId>org.junit.jupiter</groupId>
+            <artifactId>junit-jupiter-params</artifactId>
+            <scope>provided</scope>
+        </dependency>
+    </dependencies>
+</project>
diff --git a/...one/bugpatterns/MethodReferenceUsage.java → ...one/bugpatterns/MethodReferenceUsage.java b/...one/bugpatterns/MethodReferenceUsage.java → ...one/bugpatterns/MethodReferenceUsage.java
@@ -38,7 +38,7 @@
 /**
  * A {@link BugChecker} that flags lambda expressions that can be replaced with method references.
  *
- * @see IsInstanceLambdaUsage
+ * <p>See the `IsInstanceLambdaUsage` check in the `error-prone-contrib` module.
  */
 // XXX: Other custom expressions we could rewrite:
 // - `a -> "str" + a` to `"str"::concat`. But only if `str` is provably non-null.

diff --git a/...bugpatterns/MethodReferenceUsageTest.java → ...bugpatterns/MethodReferenceUsageTest.java b/...bugpatterns/MethodReferenceUsageTest.java → ...bugpatterns/MethodReferenceUsageTest.java
diff --git a/pom.xml b/pom.xml
@@ -41,6 +41,7 @@
     <modules>
         <module>documentation-support</module>
         <module>error-prone-contrib</module>
+        <module>error-prone-experimental</module>
         <module>error-prone-utils</module>
         <module>refaster-compiler</module>
         <module>refaster-runner</module>
@@ -256,6 +257,11 @@
                 <artifactId>error-prone-contrib</artifactId>
                 <version>${project.version}</version>
             </dependency>
+            <dependency>
+                <groupId>${project.groupId}</groupId>
+                <artifactId>error-prone-experimental</artifactId>
+                <version>${project.version}</version>
+            </dependency>
             <dependency>
                 <groupId>${project.groupId}</groupId>
                 <artifactId>error-prone-utils</artifactId>
@@ -1607,6 +1613,11 @@
                                     <artifactId>error-prone-contrib</artifactId>
                                     <version>${project.version}</version>
                                 </path>
+                                <path>
+                                    <groupId>${project.groupId}</groupId>
+                                    <artifactId>error-prone-experimental</artifactId>
+                                    <version>${project.version}</version>
+                                </path>
                                 <path>
                                     <groupId>${project.groupId}</groupId>
                                     <artifactId>refaster-runner</artifactId>