From f08299b134bbbc27db23deeea48231fad0c6af9f Mon Sep 17 00:00:00 2001
From: Liam Newman <bitwiseman@gmail.com>
Date: Fri, 26 Feb 2021 14:44:50 -0800
Subject: [PATCH] Only run spotless:check as part of the lifecyle

Users can run spotless:apply as they see fit.
---
 .gitattributes                   |  1 +
 .github/PULL_REQUEST_TEMPLATE.md |  3 +--
 pom.xml                          | 26 ++++++++++++++++++--------
 3 files changed, 20 insertions(+), 10 deletions(-)
 create mode 100644 .gitattributes

diff --git a/.gitattributes b/.gitattributes
new file mode 100644
index 000000000..f7bd70834
--- /dev/null
+++ b/.gitattributes
@@ -0,0 +1 @@
+*.java text eol=lf
diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md
index 3946ae598..a4adf1abf 100644
--- a/.github/PULL_REQUEST_TEMPLATE.md
+++ b/.github/PULL_REQUEST_TEMPLATE.md
@@ -7,8 +7,7 @@ We love getting PRs, but we hate asking people for the same basic changes every
 - [ ] Push your changes to a branch other than `master`. Create your PR from that branch.    
 - [ ] Add JavaDocs and other comments
 - [ ] Write tests that run and pass in CI. See [CONTRIBUTING.md](CONTRIBUTING.md) for details on how to capture snapshot data.
-- [ ] Run `mvn clean compile` locally. This may reformat your code, commit those changes.
-- [ ] Run `mvn -D enable-ci clean install site` locally. If this command doesn't succeed, your change will not pass CI.
+- [ ] Run `mvn -D enable-ci clean install site` locally. If this command doesn't succeed, your change will not pass CI. 
 
 # When creating a PR: 
 
diff --git a/pom.xml b/pom.xml
index ede281b8d..428b56e08 100644
--- a/pom.xml
+++ b/pom.xml
@@ -39,7 +39,6 @@
     <hamcrest.version>2.2</hamcrest.version>
     <okhttp3.version>4.4.1</okhttp3.version>
     <okio.version>2.5.0</okio.version>
-    <spotless-maven-plugin.goal>apply</spotless-maven-plugin.goal>
     <!-- Using this as the minimum bar for code coverage.  Adding methods without covering them will fail this. -->
     <jacoco.coverage.target.bundle.method>0.70</jacoco.coverage.target.bundle.method>
     <jacoco.coverage.target.class.method>0.50</jacoco.coverage.target.class.method>
@@ -325,13 +324,14 @@
       <plugin>
         <groupId>com.diffplug.spotless</groupId>
         <artifactId>spotless-maven-plugin</artifactId>
-        <version>2.7.0</version>
+        <version>2.8.1</version>
         <executions>
           <execution>
             <id>spotless-check</id>
-            <phase>process-sources</phase>
+            <!-- runs in verify phase by default -->
             <goals>
-              <goal>${spotless-maven-plugin.goal}</goal>
+              <!-- can be disabled using -Dspotless.check.skip=true -->
+              <goal>check</goal>
             </goals>
           </execution>
         </executions>
@@ -574,7 +574,6 @@
       <properties>
         <!-- Only fail code coverage on non-windows machines -->
         <jacoco.haltOnFailure>true</jacoco.haltOnFailure>
-        <spotless-maven-plugin.goal>check</spotless-maven-plugin.goal>
       </properties>
     </profile>
     <profile>
@@ -590,14 +589,25 @@
             <groupId>org.jacoco</groupId>
             <artifactId>jacoco-maven-plugin</artifactId>
           </plugin>
+          <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>
     <profile>
       <id>release</id>
-      <properties>
-        <spotless-maven-plugin.goal>check</spotless-maven-plugin.goal>
-      </properties>
       <build>
         <plugins>
           <plugin>