update spotless to 7.2 and reformat

.github/workflows/beam_PreCommit_Spotless.yml

@@ -77,6 +77,8 @@ jobs:
       github.event.comment.body == 'Run Spotless PreCommit'
     steps:
       - uses: actions/checkout@v6
+        with:
+          fetch-depth: 0
       - name: Setup repository
         uses: ./.github/actions/setup-action
         with:
@@ -85,11 +87,14 @@ jobs:
           github_job: ${{ matrix.job_name }} (${{ matrix.job_phrase }})
       - name: Setup environment
         uses: ./.github/actions/setup-environment-action
+        with:
+          java-version: '17'
       - name: run Spotless PreCommit script
         uses: ./.github/actions/gradle-command-self-hosted-action
         with:
           gradle-command: spotlessCheck checkStyleMain checkStyleTest :buildSrc:spotlessCheck
-          arguments: -PdisableSpotlessApply
+          arguments: -PdisableSpotlessApply ${{ github.event_name == 'schedule' && '-PdisableSpotlessRatchet=true' || '' }}
+          max-workers: 4
       - name: Upload test report
         uses: actions/upload-artifact@v7
         with:

.test-infra/metrics/build.gradle

@@ -42,7 +42,8 @@ dependencies {
 task testMetricsStack {
   doLast {
     // TODO(BEAM-5837): Add some actual validation of the metrics stack
-    println "Hello world!" }
+    println "Hello world!"
+  }
 }
 
 // Create an empty file and set DOCKER_CONFIG to ignore bad config file on Jenkins nodes. BEAM-7405

.test-infra/mock-apis/src/main/java/org/apache/beam/testinfra/mockapis/echo/v1/Echo.java

@@ -54,6 +54,7 @@ public interface EchoRequestOrBuilder
      * @return The id.
      */
     java.lang.String getId();
+
     /**
      * <code>string id = 1 [json_name = "id"];</code>
      *
@@ -68,6 +69,7 @@ public interface EchoRequestOrBuilder
      */
     com.google.protobuf.ByteString getPayload();
   }
+
   /**
    *
    *
@@ -82,6 +84,7 @@ public static final class EchoRequest extends com.google.protobuf.GeneratedMessa
       // @@protoc_insertion_point(message_implements:proto.echo.v1.EchoRequest)
       EchoRequestOrBuilder {
     private static final long serialVersionUID = 0L;
+
     // Use EchoRequest.newBuilder() to construct.
     private EchoRequest(com.google.protobuf.GeneratedMessageV3.Builder<?> builder) {
       super(builder);
@@ -122,6 +125,7 @@ public static final com.google.protobuf.Descriptors.Descriptor getDescriptor() {
 
     @SuppressWarnings("serial")
     private volatile java.lang.Object id_ = "";
+
     /**
      * <code>string id = 1 [json_name = "id"];</code>
      *
@@ -139,6 +143,7 @@ public java.lang.String getId() {
         return s;
       }
     }
+
     /**
      * <code>string id = 1 [json_name = "id"];</code>
      *
@@ -159,6 +164,7 @@ public com.google.protobuf.ByteString getIdBytes() {
 
     public static final int PAYLOAD_FIELD_NUMBER = 2;
     private com.google.protobuf.ByteString payload_ = com.google.protobuf.ByteString.EMPTY;
+
     /**
      * <code>bytes payload = 2 [json_name = "payload"];</code>
      *
@@ -339,6 +345,7 @@ protected Builder newBuilderForType(
       Builder builder = new Builder(parent);
       return builder;
     }
+
     /**
      *
      *
@@ -542,6 +549,7 @@ public Builder mergeFrom(
       private int bitField0_;
 
       private java.lang.Object id_ = "";
+
       /**
        * <code>string id = 1 [json_name = "id"];</code>
        *
@@ -558,6 +566,7 @@ public java.lang.String getId() {
           return (java.lang.String) ref;
         }
       }
+
       /**
        * <code>string id = 1 [json_name = "id"];</code>
        *
@@ -574,6 +583,7 @@ public com.google.protobuf.ByteString getIdBytes() {
           return (com.google.protobuf.ByteString) ref;
         }
       }
+
       /**
        * <code>string id = 1 [json_name = "id"];</code>
        *
@@ -589,6 +599,7 @@ public Builder setId(java.lang.String value) {
         onChanged();
         return this;
       }
+
       /**
        * <code>string id = 1 [json_name = "id"];</code>
        *
@@ -600,6 +611,7 @@ public Builder clearId() {
         onChanged();
         return this;
       }
+
       /**
        * <code>string id = 1 [json_name = "id"];</code>
        *
@@ -618,6 +630,7 @@ public Builder setIdBytes(com.google.protobuf.ByteString value) {
       }
 
       private com.google.protobuf.ByteString payload_ = com.google.protobuf.ByteString.EMPTY;
+
       /**
        * <code>bytes payload = 2 [json_name = "payload"];</code>
        *
@@ -627,6 +640,7 @@ public Builder setIdBytes(com.google.protobuf.ByteString value) {
       public com.google.protobuf.ByteString getPayload() {
         return payload_;
       }
+
       /**
        * <code>bytes payload = 2 [json_name = "payload"];</code>
        *
@@ -642,6 +656,7 @@ public Builder setPayload(com.google.protobuf.ByteString value) {
         onChanged();
         return this;
       }
+
       /**
        * <code>bytes payload = 2 [json_name = "payload"];</code>
        *
@@ -730,6 +745,7 @@ public interface EchoResponseOrBuilder
      * @return The id.
      */
     java.lang.String getId();
+
     /**
      * <code>string id = 1 [json_name = "id"];</code>
      *
@@ -744,6 +760,7 @@ public interface EchoResponseOrBuilder
      */
     com.google.protobuf.ByteString getPayload();
   }
+
   /**
    *
    *
@@ -758,6 +775,7 @@ public static final class EchoResponse extends com.google.protobuf.GeneratedMess
       // @@protoc_insertion_point(message_implements:proto.echo.v1.EchoResponse)
       EchoResponseOrBuilder {
     private static final long serialVersionUID = 0L;
+
     // Use EchoResponse.newBuilder() to construct.
     private EchoResponse(com.google.protobuf.GeneratedMessageV3.Builder<?> builder) {
       super(builder);
@@ -798,6 +816,7 @@ public static final com.google.protobuf.Descriptors.Descriptor getDescriptor() {
 
     @SuppressWarnings("serial")
     private volatile java.lang.Object id_ = "";
+
     /**
      * <code>string id = 1 [json_name = "id"];</code>
      *
@@ -815,6 +834,7 @@ public java.lang.String getId() {
         return s;
       }
     }
+
     /**
      * <code>string id = 1 [json_name = "id"];</code>
      *
@@ -835,6 +855,7 @@ public com.google.protobuf.ByteString getIdBytes() {
 
     public static final int PAYLOAD_FIELD_NUMBER = 2;
     private com.google.protobuf.ByteString payload_ = com.google.protobuf.ByteString.EMPTY;
+
     /**
      * <code>bytes payload = 2 [json_name = "payload"];</code>
      *
@@ -1015,6 +1036,7 @@ protected Builder newBuilderForType(
       Builder builder = new Builder(parent);
       return builder;
     }
+
     /**
      *
      *
@@ -1218,6 +1240,7 @@ public Builder mergeFrom(
       private int bitField0_;
 
       private java.lang.Object id_ = "";
+
       /**
        * <code>string id = 1 [json_name = "id"];</code>
        *
@@ -1234,6 +1257,7 @@ public java.lang.String getId() {
           return (java.lang.String) ref;
         }
       }
+
       /**
        * <code>string id = 1 [json_name = "id"];</code>
        *
@@ -1250,6 +1274,7 @@ public com.google.protobuf.ByteString getIdBytes() {
           return (com.google.protobuf.ByteString) ref;
         }
       }
+
       /**
        * <code>string id = 1 [json_name = "id"];</code>
        *
@@ -1265,6 +1290,7 @@ public Builder setId(java.lang.String value) {
         onChanged();
         return this;
       }
+
       /**
        * <code>string id = 1 [json_name = "id"];</code>
        *
@@ -1276,6 +1302,7 @@ public Builder clearId() {
         onChanged();
         return this;
       }
+
       /**
        * <code>string id = 1 [json_name = "id"];</code>
        *
@@ -1294,6 +1321,7 @@ public Builder setIdBytes(com.google.protobuf.ByteString value) {
       }
 
       private com.google.protobuf.ByteString payload_ = com.google.protobuf.ByteString.EMPTY;
+
       /**
        * <code>bytes payload = 2 [json_name = "payload"];</code>
        *
@@ -1303,6 +1331,7 @@ public Builder setIdBytes(com.google.protobuf.ByteString value) {
       public com.google.protobuf.ByteString getPayload() {
         return payload_;
       }
+
       /**
        * <code>bytes payload = 2 [json_name = "payload"];</code>
        *
@@ -1318,6 +1347,7 @@ public Builder setPayload(com.google.protobuf.ByteString value) {
         onChanged();
         return this;
       }
+
       /**
        * <code>bytes payload = 2 [json_name = "payload"];</code>
        *

buildSrc/build.gradle.kts

@@ -32,7 +32,7 @@ val useMirror = isCi && !mavenCentralMirrorUrl.isNullOrBlank()
 plugins {
   `java-gradle-plugin`
   groovy
-  id("com.diffplug.spotless") version "5.6.1"
+  id("com.diffplug.spotless") version "7.2.1"
 }
 
 // Define the set of repositories required to fetch and enable plugins.
@@ -68,7 +68,7 @@ dependencies {
 
   runtimeOnly("com.google.protobuf:protobuf-gradle-plugin:0.8.13")                                         // Enable proto code generation
   runtimeOnly("com.github.davidmc24.gradle.plugin:gradle-avro-plugin:1.9.1")                               // Enable Avro code generation. Version 1.1.0 is the last supporting avro 1.10.2
-  runtimeOnly("com.diffplug.spotless:spotless-plugin-gradle:5.6.1")                                        // Enable a code formatting plugin
+  runtimeOnly("com.diffplug.spotless:spotless-plugin-gradle:7.2.1")                                        // Enable a code formatting plugin
   runtimeOnly("gradle.plugin.com.dorongold.plugins:task-tree:1.5")                                         // Adds a 'taskTree' task to print task dependency tree
   runtimeOnly("net.linguica.gradle:maven-settings-plugin:0.5")
   runtimeOnly("gradle.plugin.io.pry.gradle.offline_dependencies:gradle-offline-dependencies-plugin:0.5.0") // Enable creating an offline repository

buildSrc/src/main/groovy/org/apache/beam/gradle/BeamDockerPlugin.groovy

@@ -185,7 +185,8 @@ class BeamDockerPlugin implements Plugin<Project> {
           [
             tagName: tagName,
             tagTask: {
-              -> tagName }
+              -> tagName
+            }
           ]
         ]
       }
@@ -201,7 +202,8 @@ class BeamDockerPlugin implements Plugin<Project> {
           tags[taskName] = [
             tagName: unresolvedTagName,
             tagTask: {
-              -> computeName(ext.name, unresolvedTagName) }
+              -> computeName(ext.name, unresolvedTagName)
+            }
           ]
         }
       }