Apply review

spring-projects · Jan 2, 2025 · a8ddf17 · a8ddf17
1 parent d9eee35
commit a8ddf17
Showing 1 changed file with 9 additions and 6 deletions.
diff --git a/spring-batch-bigquery/pom.xml b/spring-batch-bigquery/pom.xml
@@ -72,7 +72,13 @@
             <groupId>org.springframework.batch</groupId>
             <artifactId>spring-batch-core</artifactId>
         </dependency>
+
         <!-- Test -->
+        <dependency>
+            <groupId>ch.qos.logback</groupId>
+            <artifactId>logback-classic</artifactId>
+            <scope>test</scope>
+        </dependency>
         <dependency>
             <groupId>org.junit.jupiter</groupId>
             <artifactId>junit-jupiter-api</artifactId>
@@ -83,12 +89,6 @@
             <artifactId>mockito-core</artifactId>
             <scope>test</scope>
         </dependency>
-        <dependency>
-            <groupId>ch.qos.logback</groupId>
-            <artifactId>logback-classic</artifactId>
-            <version>${logback.version}</version>
-            <scope>test</scope>
-        </dependency>
         <dependency>
             <groupId>org.testcontainers</groupId>
             <artifactId>junit-jupiter</artifactId>
@@ -98,6 +98,7 @@
 
     <build>
         <plugins>
+            <!-- Generate javadoc and source jars -->
             <plugin>
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-javadoc-plugin</artifactId>
@@ -122,6 +123,7 @@
                     </execution>
                 </executions>
             </plugin>
+            <!-- Runs tests -->
             <plugin>
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-surefire-plugin</artifactId>
@@ -133,6 +135,7 @@
                     </includes>
                 </configuration>
             </plugin>
+            <!-- Generates a flattened version of the pom.xml, used instead of the original -->
             <plugin>
                 <groupId>org.codehaus.mojo</groupId>
                 <artifactId>flatten-maven-plugin</artifactId>