Merge pull request #392 from bitwiseman/task/formatting

Enable autoformatting
jenkinsci · Mar 4, 2021 · d3e3216 · d3e3216
2 parents 8093b3d + 6ddecf8
commit d3e3216
Show file tree

Hide file tree

Showing 94 changed files with 22,647 additions and 21,091 deletions.
diff --git a/pom.xml b/pom.xml
@@ -135,6 +135,63 @@
             </dependency>
         </dependencies>
     </dependencyManagement>
+    <build>
+        <plugins>
+            <plugin>
+                <groupId>com.diffplug.spotless</groupId>
+                <artifactId>spotless-maven-plugin</artifactId>
+                <version>2.8.1</version>
+                <executions>
+                    <execution>
+                        <id>spotless-check</id>
+                        <!-- runs in verify phase by default -->
+                        <goals>
+                            <!-- can be disabled using -Dspotless.check.skip=true -->
+                            <goal>check</goal>
+                        </goals>
+                    </execution>
+                </executions>
+                <configuration>
+                    <java>
+                        <googleJavaFormat>
+                            <style>GOOGLE</style>
+                        </googleJavaFormat>
+                    </java>
+                </configuration>
+            </plugin>
+        </plugins>
+    </build>
+    <profiles>
+        <profile>
+            <id>ci-non-windows</id>
+            <activation>
+                <property>
+                    <name>set.changelist</name>
+                </property>
+                <os>
+                    <family>!windows</family>
+                </os>
+            </activation>
+            <build>
+                <plugins>
+                    <plugin>
+                        <groupId>com.diffplug.spotless</groupId>
+                        <artifactId>spotless-maven-plugin</artifactId>
+                        <executions>
+                            <execution>
+                                <id>spotless-check</id>
+                                <!-- In CI, run check early in the build -->
+                                <phase>process-sources</phase>
+                                <goals>
+                                    <goal>check</goal>
+                                </goals>
+                            </execution>
+                        </executions>
+                    </plugin>
+                </plugins>
+            </build>
+        </profile>
+    </profiles>
     <repositories>
         <repository>
             <id>repo.jenkins-ci.org</id>

diff --git a/...n/java/org/jenkinsci/plugins/github_branch_source/AbstractGitHubNotificationStrategy.java b/...n/java/org/jenkinsci/plugins/github_branch_source/AbstractGitHubNotificationStrategy.java
@@ -25,31 +25,29 @@
 package org.jenkinsci.plugins.github_branch_source;
 
 import hudson.model.TaskListener;
-
 import java.util.List;
 
 /**
  * Represents a strategy for constructing GitHub status notifications
+ *
  * @since 2.3.2
  */
 public abstract class AbstractGitHubNotificationStrategy {
 
-    /**
-     * Creates the list of {@link GitHubNotificationRequest} for the given context.
-     * @param notificationContext {@link GitHubNotificationContext} the context details
-     * @param listener the listener
-     * @return a list of notification requests
-     * @since 2.3.2
-     */
-    public abstract List<GitHubNotificationRequest> notifications(GitHubNotificationContext notificationContext, TaskListener listener);
+  /**
+   * Creates the list of {@link GitHubNotificationRequest} for the given context.
+   *
+   * @param notificationContext {@link GitHubNotificationContext} the context details
+   * @param listener the listener
+   * @return a list of notification requests
+   * @since 2.3.2
+   */
+  public abstract List<GitHubNotificationRequest> notifications(
+      GitHubNotificationContext notificationContext, TaskListener listener);
 
-    /**
-     * {@inheritDoc}
-     */
-    public abstract boolean equals(Object o);
+  /** {@inheritDoc} */
+  public abstract boolean equals(Object o);
 
-    /**
-     * {@inheritDoc}
-     */
-    public abstract int hashCode();
+  /** {@inheritDoc} */
+  public abstract int hashCode();
 }