From fd774a33f1564d46968c59e60016b725e6245466 Mon Sep 17 00:00:00 2001
From: Vincent Potucek <vpotucek@me.com>
Date: Mon, 1 Sep 2025 11:20:03 +0200
Subject: [PATCH 1/2] org.openrewrite.maven.BestPractices

---
 pmd.xml     | 13 ++++++++++++
 pom.xml     | 60 +++++++++++++++++++++++++++++++++++++++++++++++++++++
 rewrite.yml |  6 ++++++
 3 files changed, 79 insertions(+)
 create mode 100644 pmd.xml
 create mode 100644 rewrite.yml

diff --git a/pmd.xml b/pmd.xml
new file mode 100644
index 000000000000..ed9ff2f82dc4
--- /dev/null
+++ b/pmd.xml
@@ -0,0 +1,13 @@
+<?xml version="1.0"?>
+<ruleset name="DefinitionOfDone"
+         xmlns="http://pmd.sourceforge.net/ruleset/2.0.0"
+         xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         xsi:schemaLocation="http://pmd.sourceforge.net/ruleset/2.0.0 https://pmd.sourceforge.net/ruleset_2_0_0.xsd">
+    <description>DefinitionOfDone</description>
+    <exclude-pattern>.*/Multisets.java</exclude-pattern><!--https://github.com/pmd/pmd/issues/6028-->
+    <exclude-pattern>.*/com/google/common/reflect/A*.*</exclude-pattern>
+    <rule ref="category/java/bestpractices.xml/UnusedPrivateMethod"/>
+    <!--<rule ref="category/java/errorprone.xml/OverrideBothEqualsAndHashCodeOnComparable" />-->
+    <!--<rule ref="category/java/codestyle.xml/UnnecessaryImport" /> https://github.com/pmd/pmd/issues/6027 -->
+    <!--<rule ref="category/java/bestpractices.xml/ModifierOrder"/> https://github.com/pmd/pmd/issues/6019 -->
+</ruleset>
\ No newline at end of file
diff --git a/pom.xml b/pom.xml
index d1eaee8aae32..75bc17d75f25 100644
--- a/pom.xml
+++ b/pom.xml
@@ -130,9 +130,69 @@
           </execution>
         </executions>
       </plugin>
+      <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
+        <artifactId>maven-pmd-plugin</artifactId>
+      </plugin>
+      <plugin>
+        <groupId>org.openrewrite.maven</groupId>
+        <artifactId>rewrite-maven-plugin</artifactId>
+      </plugin>
     </plugins>
     <pluginManagement>
       <plugins>
+        <plugin>
+          <groupId>org.apache.maven.plugins</groupId>
+          <artifactId>maven-pmd-plugin</artifactId>
+          <version>3.27.0</version>
+          <configuration>
+            <analysisCache>true</analysisCache>
+            <failOnViolation>true</failOnViolation>
+            <includeTests>true</includeTests>
+            <printFailingErrors>true</printFailingErrors>
+            <rulesets>
+              <ruleset>pmd.xml</ruleset>
+            </rulesets>
+          </configuration>
+          <executions>
+            <execution>
+              <phase>verify</phase>
+              <goals>
+                <goal>check</goal>
+              </goals>
+            </execution>
+          </executions>
+        </plugin>
+        <plugin>
+          <groupId>org.openrewrite.maven</groupId>
+          <artifactId>rewrite-maven-plugin</artifactId>
+          <version>6.15.0</version>
+          <configuration>
+            <activeRecipes>
+              <recipe>DefinitionOfDone</recipe>
+            </activeRecipes>
+            <exclusions>
+              <exclusion>package-info.java</exclusion>
+            </exclusions>
+            <exportDatatables>true</exportDatatables>
+            <failOnDryRunResults>true</failOnDryRunResults>
+          </configuration>
+          <executions>
+            <execution>
+              <phase>verify</phase>
+              <goals>
+                <goal>dryRun</goal>
+              </goals>
+            </execution>
+          </executions>
+          <dependencies>
+            <dependency>
+              <groupId>org.openrewrite.recipe</groupId>
+              <artifactId>rewrite-static-analysis</artifactId>
+              <version>2.16.0</version>
+            </dependency>
+          </dependencies>
+        </plugin>
         <plugin>
           <artifactId>maven-antrun-plugin</artifactId>
           <version>1.6</version>
diff --git a/rewrite.yml b/rewrite.yml
new file mode 100644
index 000000000000..de2ff2aadc29
--- /dev/null
+++ b/rewrite.yml
@@ -0,0 +1,6 @@
+type: specs.openrewrite.org/v1beta/recipe
+name: DefinitionOfDone
+displayName: Definition of Done
+description: Automatically cleanup code to comply checkstyle convention.
+recipeList:
+  - org.openrewrite.maven.BestPractices

From 881bbf09fbda29d0b96c95cf565e7fde7071b1d9 Mon Sep 17 00:00:00 2001
From: Vincent Potucek <vpotucek@me.com>
Date: Mon, 1 Sep 2025 11:30:18 +0200
Subject: [PATCH 2/2] Run maven BestPractices

---
 guava-bom/pom.xml     |  10 +-
 guava-gwt/pom.xml     |  78 +++++++++-----
 guava-testlib/pom.xml |   6 ++
 guava-tests/pom.xml   |   6 ++
 guava/pom.xml         |  11 +-
 pmd.xml               |  13 ---
 pom.xml               | 234 +++++++++++++++++-------------------------
 rewrite.yml           |   6 --
 8 files changed, 171 insertions(+), 193 deletions(-)
 delete mode 100644 pmd.xml
 delete mode 100644 rewrite.yml

diff --git a/guava-bom/pom.xml b/guava-bom/pom.xml
index 64bc74cb5cd0..e1cbaa8c1e79 100644
--- a/guava-bom/pom.xml
+++ b/guava-bom/pom.xml
@@ -16,11 +16,6 @@
   <url>https://github.com/google/guava</url>
   <inceptionYear>2010</inceptionYear>
 
-  <issueManagement>
-    <system>GitHub Issues</system>
-    <url>https://github.com/google/guava/issues</url>
-  </issueManagement>
-
   <licenses>
     <license>
       <name>Apache License, Version 2.0</name>
@@ -29,6 +24,11 @@
     </license>
   </licenses>
 
+  <issueManagement>
+    <system>GitHub Issues</system>
+    <url>https://github.com/google/guava/issues</url>
+  </issueManagement>
+
   <distributionManagement>
     <snapshotRepository>
       <id>sonatype-nexus-snapshots</id>
diff --git a/guava-gwt/pom.xml b/guava-gwt/pom.xml
index db9b0fedca72..adf2a40ac554 100644
--- a/guava-gwt/pom.xml
+++ b/guava-gwt/pom.xml
@@ -21,26 +21,6 @@
     <gwt.plugin.version>2.10.0</gwt.plugin.version>
     <gwt.logLevel>WARN</gwt.logLevel>
   </properties>
-  <repositories>
-    <repository>
-      <id>sonatype-google-snapshots</id>
-      <name>sonatype-google-snapshots</name>
-      <url>https://oss.sonatype.org/content/repositories/google-snapshots/</url>
-      <snapshots>
-        <enabled>true</enabled>
-      </snapshots>
-    </repository>
-  </repositories>
-  <pluginRepositories>
-    <pluginRepository>
-      <id>sonatype-snapshots</id>
-      <name>sonatype-snapshots</name>
-      <url>https://oss.sonatype.org/content/repositories/snapshots/</url>
-      <snapshots>
-        <enabled>true</enabled>
-      </snapshots>
-    </pluginRepository>
-  </pluginRepositories>
   <dependencies>
     <dependency>
       <groupId>com.google.guava</groupId>
@@ -123,6 +103,26 @@
       </exclusions>
     </dependency>
   </dependencies>
+  <repositories>
+    <repository>
+      <id>sonatype-google-snapshots</id>
+      <name>sonatype-google-snapshots</name>
+      <url>https://oss.sonatype.org/content/repositories/google-snapshots/</url>
+      <snapshots>
+        <enabled>true</enabled>
+      </snapshots>
+    </repository>
+  </repositories>
+  <pluginRepositories>
+    <pluginRepository>
+      <id>sonatype-snapshots</id>
+      <name>sonatype-snapshots</name>
+      <url>https://oss.sonatype.org/content/repositories/snapshots/</url>
+      <snapshots>
+        <enabled>true</enabled>
+      </snapshots>
+    </pluginRepository>
+  </pluginRepositories>
   <build>
     <plugins>
       <plugin>
@@ -130,9 +130,11 @@
         <artifactId>toolchains-maven-plugin</artifactId>
       </plugin>
       <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-toolchains-plugin</artifactId>
       </plugin>
       <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-compiler-plugin</artifactId>
         <executions>
           <execution>
@@ -150,6 +152,7 @@
         </executions>
       </plugin>
       <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-jar-plugin</artifactId>
         <configuration>
           <excludes>
@@ -160,6 +163,7 @@
         </configuration>
       </plugin>
       <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-source-plugin</artifactId>
         <configuration>
           <excludes>
@@ -170,6 +174,7 @@
         </configuration>
       </plugin>
       <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-javadoc-plugin</artifactId>
         <configuration>
           <sourceFileExcludes>
@@ -184,18 +189,22 @@
       </plugin>
       <!-- Disable "normal" testing, which doesn't work for GWT tests. -->
       <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-surefire-plugin</artifactId>
         <configuration>
           <skip>true</skip>
         </configuration>
       </plugin>
       <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-dependency-plugin</artifactId>
         <executions>
           <execution>
             <id>unpack-failureaccess-sources</id>
             <phase>generate-resources</phase>
-            <goals><goal>unpack-dependencies</goal></goals>
+            <goals>
+              <goal>unpack-dependencies</goal>
+            </goals>
             <configuration>
               <includeArtifactIds>failureaccess</includeArtifactIds>
               <classifier>sources</classifier>
@@ -209,7 +218,9 @@
           <execution>
             <id>unpack-guava-sources</id>
             <phase>generate-resources</phase>
-            <goals><goal>unpack-dependencies</goal></goals>
+            <goals>
+              <goal>unpack-dependencies</goal>
+            </goals>
             <configuration>
               <includeArtifactIds>guava</includeArtifactIds>
               <classifier>sources</classifier>
@@ -223,7 +234,9 @@
           <execution>
             <id>unpack-guava-testlib-sources</id>
             <phase>generate-resources</phase>
-            <goals><goal>unpack-dependencies</goal></goals>
+            <goals>
+              <goal>unpack-dependencies</goal>
+            </goals>
             <configuration>
               <includeArtifactIds>guava-testlib</includeArtifactIds>
               <classifier>sources</classifier>
@@ -237,7 +250,9 @@
           <execution>
             <id>unpack-guava-testlib-test-sources</id>
             <phase>generate-resources</phase>
-            <goals><goal>unpack-dependencies</goal></goals>
+            <goals>
+              <goal>unpack-dependencies</goal>
+            </goals>
             <configuration>
               <includeArtifactIds>guava-testlib</includeArtifactIds>
               <classifier>test-sources</classifier>
@@ -251,7 +266,9 @@
           <execution>
             <id>unpack-guava-test-sources</id>
             <phase>generate-resources</phase>
-            <goals><goal>unpack-dependencies</goal></goals>
+            <goals>
+              <goal>unpack-dependencies</goal>
+            </goals>
             <configuration>
               <includeArtifactIds>guava-tests</includeArtifactIds>
               <classifier>test-sources</classifier>
@@ -297,12 +314,15 @@
         </dependencies>
       </plugin>
       <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-antrun-plugin</artifactId>
         <executions>
           <execution>
             <id>copy-gwt-files</id>
             <phase>generate-resources</phase>
-            <goals><goal>run</goal></goals>
+            <goals>
+              <goal>run</goal>
+            </goals>
             <configuration>
               <target name="copy-gwt-resources">
                 <delete dir="${project.build.directory}/guava-gwt-sources" quiet="true" />
@@ -418,8 +438,10 @@
               <extraJvmArgs>-Xms3500m -Xmx3500m -Xss1024k</extraJvmArgs>
               <sourceLevel>1.8</sourceLevel>
               <!-- Keep these timeouts very large because, if we hit the timeout, the tests silently pass :( -->
-              <testTimeOut>86400 <!-- seconds --></testTimeOut>
-              <testMethodTimeout>1440 <!-- minutes --></testMethodTimeout>
+              <testTimeOut>86400 
+                <!-- seconds --></testTimeOut>
+              <testMethodTimeout>1440 
+                <!-- minutes --></testMethodTimeout>
               <!-- Presumably we want watchFileChanges=false here, since we want it for compile: -->
               <extraJvmArgs>-Dgwt.watchFileChanges=false</extraJvmArgs>
             </configuration>
diff --git a/guava-testlib/pom.xml b/guava-testlib/pom.xml
index 8fe4208cc7f9..0a90d448b371 100644
--- a/guava-testlib/pom.xml
+++ b/guava-testlib/pom.xml
@@ -67,9 +67,11 @@
         <artifactId>toolchains-maven-plugin</artifactId>
       </plugin>
       <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-toolchains-plugin</artifactId>
       </plugin>
       <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-compiler-plugin</artifactId>
         <executions>
           <execution>
@@ -112,6 +114,7 @@
         </executions>
       </plugin>
       <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-source-plugin</artifactId>
         <executions>
           <execution>
@@ -122,6 +125,7 @@
         </executions>
       </plugin>
       <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-jar-plugin</artifactId>
         <executions>
           <execution>
@@ -135,9 +139,11 @@
         <artifactId>animal-sniffer-maven-plugin</artifactId>
       </plugin>
       <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-javadoc-plugin</artifactId>
       </plugin>
       <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-surefire-plugin</artifactId>
       </plugin>
     </plugins>
diff --git a/guava-tests/pom.xml b/guava-tests/pom.xml
index fd0d5dd6b7ef..a00ad5f633f6 100644
--- a/guava-tests/pom.xml
+++ b/guava-tests/pom.xml
@@ -73,12 +73,15 @@
         <artifactId>toolchains-maven-plugin</artifactId>
       </plugin>
       <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-toolchains-plugin</artifactId>
       </plugin>
       <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-compiler-plugin</artifactId>
       </plugin>
       <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-source-plugin</artifactId>
         <executions>
           <execution>
@@ -89,9 +92,11 @@
         </executions>
       </plugin>
       <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-surefire-plugin</artifactId>
       </plugin>
       <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-jar-plugin</artifactId>
         <executions>
           <execution>
@@ -103,6 +108,7 @@
       <plugin>
         <groupId>org.sonatype.central</groupId>
         <artifactId>central-publishing-maven-plugin</artifactId>
+        <version>0.8.0</version>
         <configuration>
           <skipPublishing>true</skipPublishing>
         </configuration>
diff --git a/guava/pom.xml b/guava/pom.xml
index 5683efcd0a10..17d8bd066337 100644
--- a/guava/pom.xml
+++ b/guava/pom.xml
@@ -11,12 +11,12 @@
   <artifactId>guava</artifactId>
   <packaging>bundle</packaging>
   <name>Guava: Google Core Libraries for Java</name>
-  <url>https://github.com/google/guava</url>
   <description>
     Guava is a suite of core and expanded libraries that include
     utility classes, Google's collections, I/O classes, and
     much more.
   </description>
+  <url>https://github.com/google/guava</url>
   <dependencies>
     <dependency>
       <groupId>com.google.guava</groupId>
@@ -46,7 +46,8 @@
       <resource>
         <directory>..</directory>
         <includes>
-          <include>LICENSE</include> <!-- copied from the parent pom because I couldn't figure out a way to make combine.children="append" work -->
+          <include>LICENSE</include>
+          <!-- copied from the parent pom because I couldn't figure out a way to make combine.children="append" work -->
           <include>proguard/*</include>
         </includes>
         <targetPath>META-INF</targetPath>
@@ -58,9 +59,11 @@
         <artifactId>toolchains-maven-plugin</artifactId>
       </plugin>
       <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-toolchains-plugin</artifactId>
       </plugin>
       <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-jar-plugin</artifactId>
       </plugin>
       <plugin>
@@ -98,6 +101,7 @@
         </configuration>
       </plugin>
       <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-compiler-plugin</artifactId>
         <executions>
           <execution>
@@ -140,6 +144,7 @@
         </executions>
       </plugin>
       <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-source-plugin</artifactId>
       </plugin>
       <plugin>
@@ -147,6 +152,7 @@
         <artifactId>animal-sniffer-maven-plugin</artifactId>
       </plugin>
       <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-javadoc-plugin</artifactId>
         <configuration>
           <!-- Passing `-subpackages com.google.common` breaks things, so we explicitly exclude everything else instead. -->
@@ -202,6 +208,7 @@
         </configuration>
       </plugin>
       <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-resources-plugin</artifactId>
         <executions>
           <execution>
diff --git a/pmd.xml b/pmd.xml
deleted file mode 100644
index ed9ff2f82dc4..000000000000
--- a/pmd.xml
+++ /dev/null
@@ -1,13 +0,0 @@
-<?xml version="1.0"?>
-<ruleset name="DefinitionOfDone"
-         xmlns="http://pmd.sourceforge.net/ruleset/2.0.0"
-         xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
-         xsi:schemaLocation="http://pmd.sourceforge.net/ruleset/2.0.0 https://pmd.sourceforge.net/ruleset_2_0_0.xsd">
-    <description>DefinitionOfDone</description>
-    <exclude-pattern>.*/Multisets.java</exclude-pattern><!--https://github.com/pmd/pmd/issues/6028-->
-    <exclude-pattern>.*/com/google/common/reflect/A*.*</exclude-pattern>
-    <rule ref="category/java/bestpractices.xml/UnusedPrivateMethod"/>
-    <!--<rule ref="category/java/errorprone.xml/OverrideBothEqualsAndHashCodeOnComparable" />-->
-    <!--<rule ref="category/java/codestyle.xml/UnnecessaryImport" /> https://github.com/pmd/pmd/issues/6027 -->
-    <!--<rule ref="category/java/bestpractices.xml/ModifierOrder"/> https://github.com/pmd/pmd/issues/6019 -->
-</ruleset>
\ No newline at end of file
diff --git a/pom.xml b/pom.xml
index 75bc17d75f25..7df8830fa467 100644
--- a/pom.xml
+++ b/pom.xml
@@ -1,7 +1,7 @@
 <?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 http://maven.apache.org/maven-v4_0_0.xsd">
+  xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+  xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
 
   <modelVersion>4.0.0</modelVersion>
   <groupId>com.google.guava</groupId>
@@ -11,6 +11,59 @@
   <name>Guava Maven Parent</name>
   <description>Parent for guava artifacts</description>
   <url>https://github.com/google/guava</url>
+  <inceptionYear>2010</inceptionYear>
+  <licenses>
+    <license>
+      <name>Apache License, Version 2.0</name>
+      <url>http://www.apache.org/licenses/LICENSE-2.0.txt</url>
+      <distribution>repo</distribution>
+    </license>
+  </licenses>
+  <developers>
+    <developer>
+      <id>kevinb9n</id>
+      <name>Kevin Bourrillion</name>
+      <email>kevinb@google.com</email>
+      <organization>Google</organization>
+      <organizationUrl>http://www.google.com</organizationUrl>
+      <roles>
+        <role>owner</role>
+        <role>developer</role>
+      </roles>
+      <timezone>-8</timezone>
+    </developer>
+  </developers>
+  <modules>
+    <module>guava</module>
+    <module>guava-bom</module>
+    <module>guava-gwt</module>
+    <module>guava-testlib</module>
+    <module>guava-tests</module>
+  </modules>
+  <scm>
+    <connection>scm:git:https://github.com/google/guava.git</connection>
+    <developerConnection>scm:git:git@github.com:google/guava.git</developerConnection>
+    <url>https://github.com/google/guava</url>
+  </scm>
+  <issueManagement>
+    <system>GitHub Issues</system>
+    <url>https://github.com/google/guava/issues</url>
+  </issueManagement>
+  <ciManagement>
+    <system>GitHub Actions</system>
+    <url>https://github.com/google/guava/actions</url>
+  </ciManagement>
+  <distributionManagement>
+    <snapshotRepository>
+      <id>sonatype-nexus-snapshots</id>
+      <url>https://central.sonatype.com/repository/maven-snapshots/</url>
+    </snapshotRepository>
+    <site>
+      <id>guava-site</id>
+      <name>Guava Documentation Site</name>
+      <url>scp://dummy.server/dontinstall/usestaging</url>
+    </site>
+  </distributionManagement>
   <properties>
     <!--
     When building Guava, you can pass (e.g.) `-Dsurefire.toolchain.version=21` to change which version to run tests under.
@@ -45,48 +98,30 @@
     <otherVariant.jvmEnvironment>android</otherVariant.jvmEnvironment>
     <otherVariant.jvmEnvironmentVariantName>android</otherVariant.jvmEnvironmentVariantName>
   </properties>
-  <issueManagement>
-    <system>GitHub Issues</system>
-    <url>https://github.com/google/guava/issues</url>
-  </issueManagement>
-  <inceptionYear>2010</inceptionYear>
-  <licenses>
-    <license>
-      <name>Apache License, Version 2.0</name>
-      <url>http://www.apache.org/licenses/LICENSE-2.0.txt</url>
-      <distribution>repo</distribution>
-    </license>
-  </licenses>
-  <scm>
-    <connection>scm:git:https://github.com/google/guava.git</connection>
-    <developerConnection>scm:git:git@github.com:google/guava.git</developerConnection>
-    <url>https://github.com/google/guava</url>
-  </scm>
-  <developers>
-    <developer>
-      <id>kevinb9n</id>
-      <name>Kevin Bourrillion</name>
-      <email>kevinb@google.com</email>
-      <organization>Google</organization>
-      <organizationUrl>http://www.google.com</organizationUrl>
-      <roles>
-        <role>owner</role>
-        <role>developer</role>
-      </roles>
-      <timezone>-8</timezone>
-    </developer>
-  </developers>
-  <ciManagement>
-    <system>GitHub Actions</system>
-    <url>https://github.com/google/guava/actions</url>
-  </ciManagement>
-  <modules>
-    <module>guava</module>
-    <module>guava-bom</module>
-    <module>guava-gwt</module>
-    <module>guava-testlib</module>
-    <module>guava-tests</module>
-  </modules>
+  <dependencyManagement>
+    <dependencies>
+      <dependency>
+        <groupId>org.jspecify</groupId>
+        <artifactId>jspecify</artifactId>
+        <version>${jspecify.version}</version>
+      </dependency>
+      <dependency>
+        <groupId>com.google.errorprone</groupId>
+        <artifactId>error_prone_annotations</artifactId>
+        <version>${errorprone.version}</version>
+      </dependency>
+      <dependency>
+        <groupId>com.google.j2objc</groupId>
+        <artifactId>j2objc-annotations</artifactId>
+        <version>${j2objc.version}</version>
+      </dependency>
+      <!--
+      We moved away from using dependencyManagement for test-only deps because of https://github.com/google/guava/issues/6654.
+      We could probably have resumed it after https://github.com/google/guava/pull/6664.
+      But it's always weird that published poms reference test-only libraries at all, so I'm not in any rush to do so.
+      -->
+    </dependencies>
+  </dependencyManagement>
   <build>
     <!-- Handle where Guava deviates from Maven defaults -->
     <sourceDirectory>src</sourceDirectory>
@@ -110,6 +145,7 @@
     </testResources>
     <plugins>
       <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-enforcer-plugin</artifactId>
         <executions>
           <execution>
@@ -130,74 +166,16 @@
           </execution>
         </executions>
       </plugin>
-      <plugin>
-        <groupId>org.apache.maven.plugins</groupId>
-        <artifactId>maven-pmd-plugin</artifactId>
-      </plugin>
-      <plugin>
-        <groupId>org.openrewrite.maven</groupId>
-        <artifactId>rewrite-maven-plugin</artifactId>
-      </plugin>
     </plugins>
     <pluginManagement>
       <plugins>
         <plugin>
           <groupId>org.apache.maven.plugins</groupId>
-          <artifactId>maven-pmd-plugin</artifactId>
-          <version>3.27.0</version>
-          <configuration>
-            <analysisCache>true</analysisCache>
-            <failOnViolation>true</failOnViolation>
-            <includeTests>true</includeTests>
-            <printFailingErrors>true</printFailingErrors>
-            <rulesets>
-              <ruleset>pmd.xml</ruleset>
-            </rulesets>
-          </configuration>
-          <executions>
-            <execution>
-              <phase>verify</phase>
-              <goals>
-                <goal>check</goal>
-              </goals>
-            </execution>
-          </executions>
-        </plugin>
-        <plugin>
-          <groupId>org.openrewrite.maven</groupId>
-          <artifactId>rewrite-maven-plugin</artifactId>
-          <version>6.15.0</version>
-          <configuration>
-            <activeRecipes>
-              <recipe>DefinitionOfDone</recipe>
-            </activeRecipes>
-            <exclusions>
-              <exclusion>package-info.java</exclusion>
-            </exclusions>
-            <exportDatatables>true</exportDatatables>
-            <failOnDryRunResults>true</failOnDryRunResults>
-          </configuration>
-          <executions>
-            <execution>
-              <phase>verify</phase>
-              <goals>
-                <goal>dryRun</goal>
-              </goals>
-            </execution>
-          </executions>
-          <dependencies>
-            <dependency>
-              <groupId>org.openrewrite.recipe</groupId>
-              <artifactId>rewrite-static-analysis</artifactId>
-              <version>2.16.0</version>
-            </dependency>
-          </dependencies>
-        </plugin>
-        <plugin>
           <artifactId>maven-antrun-plugin</artifactId>
           <version>1.6</version>
         </plugin>
         <plugin>
+          <groupId>org.apache.maven.plugins</groupId>
           <artifactId>maven-compiler-plugin</artifactId>
           <version>3.13.0</version>
           <configuration>
@@ -276,18 +254,22 @@
           </executions>
         </plugin>
         <plugin>
+          <groupId>org.apache.maven.plugins</groupId>
           <artifactId>maven-dependency-plugin</artifactId>
           <version>3.1.1</version>
         </plugin>
         <plugin>
+          <groupId>org.apache.maven.plugins</groupId>
           <artifactId>maven-enforcer-plugin</artifactId>
           <version>3.0.0-M3</version>
         </plugin>
         <plugin>
+          <groupId>org.apache.maven.plugins</groupId>
           <artifactId>maven-install-plugin</artifactId>
           <version>3.1.3</version>
         </plugin>
         <plugin>
+          <groupId>org.apache.maven.plugins</groupId>
           <artifactId>maven-jar-plugin</artifactId>
           <version>3.4.2</version>
           <configuration>
@@ -314,6 +296,7 @@
           </dependencies>
         </plugin>
         <plugin>
+          <groupId>org.apache.maven.plugins</groupId>
           <artifactId>maven-javadoc-plugin</artifactId>
           <version>3.11.2</version>
           <configuration>
@@ -331,15 +314,19 @@
           <executions>
             <execution>
               <id>attach-docs</id>
-              <goals><goal>jar</goal></goals>
+              <goals>
+                <goal>jar</goal>
+              </goals>
             </execution>
           </executions>
         </plugin>
         <plugin>
+          <groupId>org.apache.maven.plugins</groupId>
           <artifactId>maven-resources-plugin</artifactId>
           <version>3.3.1</version>
         </plugin>
         <plugin>
+          <groupId>org.apache.maven.plugins</groupId>
           <artifactId>maven-source-plugin</artifactId>
           <version>3.3.1</version>
           <executions>
@@ -360,6 +347,7 @@
           </dependencies>
         </plugin>
         <plugin>
+          <groupId>org.apache.maven.plugins</groupId>
           <artifactId>maven-surefire-plugin</artifactId>
           <version>3.3.1</version>
           <configuration>
@@ -386,6 +374,7 @@
           </configuration>
         </plugin>
         <plugin>
+          <groupId>org.apache.maven.plugins</groupId>
           <artifactId>maven-toolchains-plugin</artifactId>
           <version>3.2.0</version>
           <executions>
@@ -485,47 +474,13 @@
       </plugins>
     </pluginManagement>
   </build>
-  <distributionManagement>
-    <snapshotRepository>
-      <id>sonatype-nexus-snapshots</id>
-      <url>https://central.sonatype.com/repository/maven-snapshots/</url>
-    </snapshotRepository>
-    <site>
-      <id>guava-site</id>
-      <name>Guava Documentation Site</name>
-      <url>scp://dummy.server/dontinstall/usestaging</url>
-    </site>
-  </distributionManagement>
-  <dependencyManagement>
-    <dependencies>
-      <dependency>
-        <groupId>org.jspecify</groupId>
-        <artifactId>jspecify</artifactId>
-        <version>${jspecify.version}</version>
-      </dependency>
-      <dependency>
-        <groupId>com.google.errorprone</groupId>
-        <artifactId>error_prone_annotations</artifactId>
-        <version>${errorprone.version}</version>
-      </dependency>
-      <dependency>
-        <groupId>com.google.j2objc</groupId>
-        <artifactId>j2objc-annotations</artifactId>
-        <version>${j2objc.version}</version>
-      </dependency>
-      <!--
-      We moved away from using dependencyManagement for test-only deps because of https://github.com/google/guava/issues/6654.
-      We could probably have resumed it after https://github.com/google/guava/pull/6664.
-      But it's always weird that published poms reference test-only libraries at all, so I'm not in any rush to do so.
-      -->
-    </dependencies>
-  </dependencyManagement>
   <profiles>
     <profile>
       <id>sonatype-oss-release</id>
       <build>
         <plugins>
           <plugin>
+            <groupId>org.apache.maven.plugins</groupId>
             <artifactId>maven-gpg-plugin</artifactId>
             <version>3.0.1</version>
             <executions>
@@ -600,6 +555,7 @@
             </executions>
           </plugin>
           <plugin>
+            <groupId>org.apache.maven.plugins</groupId>
             <artifactId>maven-toolchains-plugin</artifactId>
             <executions>
               <execution>
diff --git a/rewrite.yml b/rewrite.yml
deleted file mode 100644
index de2ff2aadc29..000000000000
--- a/rewrite.yml
+++ /dev/null
@@ -1,6 +0,0 @@
-type: specs.openrewrite.org/v1beta/recipe
-name: DefinitionOfDone
-displayName: Definition of Done
-description: Automatically cleanup code to comply checkstyle convention.
-recipeList:
-  - org.openrewrite.maven.BestPractices