From 892b2b44adff2f55a62e0cc0ed3d60c1738b83d1 Mon Sep 17 00:00:00 2001
From: svc-excavator-bot <svc-excavator-bot@palantir.com>
Date: Thu, 26 Sep 2019 21:20:44 +0000
Subject: [PATCH] Excavator: Upgrades Baseline to the latest version

---
 .baseline/checkstyle/checkstyle.xml                         | 2 +-
 .baseline/spotless/eclipse.xml                              | 3 ++-
 build.gradle                                                | 2 +-
 .../palantir/gradle/conjure/CompileConjurePythonTask.java   | 2 +-
 .../gradle/conjure/CompileConjureTypeScriptTask.java        | 2 +-
 .../com/palantir/gradle/conjure/ConjureGeneratorTask.java   | 6 +++---
 .../java/com/palantir/gradle/conjure/ConjurePlugin.java     | 2 +-
 .../com/palantir/gradle/conjure/ExtractExecutableTask.java  | 6 +++---
 8 files changed, 13 insertions(+), 12 deletions(-)

diff --git a/.baseline/checkstyle/checkstyle.xml b/.baseline/checkstyle/checkstyle.xml
index 268ebe3fc..f541b63e2 100644
--- a/.baseline/checkstyle/checkstyle.xml
+++ b/.baseline/checkstyle/checkstyle.xml
@@ -469,7 +469,7 @@
         <module name="NestedTryDepth"/> <!-- Java Coding Guide: Try/catch blocks: never nested -->
         <module name="NonEmptyAtclauseDescription"/> <!-- Java Style Guide: At-clauses -->
         <module name="ParameterName"> <!-- Java Style Guide: Parameter names -->
-            <property name="format" value="^[a-z][a-zA-Z0-9]+$"/>
+            <property name="format" value="^_?[a-z][a-zA-Z0-9]+$"/>
             <message key="name.invalidPattern" value="Parameter name ''{0}'' must match pattern ''{1}''."/>
         </module>
         <module name="SingleLineJavadoc"/> <!-- Java Style Guide: General form -->
diff --git a/.baseline/spotless/eclipse.xml b/.baseline/spotless/eclipse.xml
index cff10b095..fe1fc71ea 100644
--- a/.baseline/spotless/eclipse.xml
+++ b/.baseline/spotless/eclipse.xml
@@ -92,6 +92,7 @@
         <setting id="org.eclipse.jdt.core.formatter.insert_space_after_closing_angle_bracket_in_type_arguments" value="do not insert"/>
         <setting id="org.eclipse.jdt.core.formatter.keep_else_statement_on_same_line" value="false"/>
         <setting id="org.eclipse.jdt.core.formatter.alignment_for_binary_expression" value="80"/>
+        <setting id="org.eclipse.jdt.core.formatter.alignment_for_string_concatenation" value="16"/>
         <setting id="org.eclipse.jdt.core.formatter.parentheses_positions_in_catch_clause" value="common_lines"/>
         <setting id="org.eclipse.jdt.core.formatter.insert_space_after_comma_in_parameterized_type_reference" value="insert"/>
         <setting id="org.eclipse.jdt.core.formatter.insert_space_before_comma_in_array_initializer" value="do not insert"/>
@@ -177,7 +178,7 @@
         <setting id="org.eclipse.jdt.core.formatter.comment.new_lines_at_javadoc_boundaries" value="true"/>
         <setting id="org.eclipse.jdt.core.formatter.brace_position_for_annotation_type_declaration" value="end_of_line"/>
         <setting id="org.eclipse.jdt.core.formatter.insert_space_before_colon_in_for" value="insert"/>
-        <setting id="org.eclipse.jdt.core.formatter.alignment_for_resources_in_try" value="48"/>
+        <setting id="org.eclipse.jdt.core.formatter.alignment_for_resources_in_try" value="80"/>
         <setting id="org.eclipse.jdt.core.formatter.use_tabs_only_for_leading_indentations" value="false"/>
         <setting id="org.eclipse.jdt.core.formatter.parentheses_positions_in_try_clause" value="common_lines"/>
         <setting id="org.eclipse.jdt.core.formatter.alignment_for_selector_in_method_invocation" value="80"/>
diff --git a/build.gradle b/build.gradle
index 85576689a..574792a16 100644
--- a/build.gradle
+++ b/build.gradle
@@ -28,7 +28,7 @@ buildscript {
     dependencies {
         classpath 'com.jfrog.bintray.gradle:gradle-bintray-plugin:1.8.4'
         classpath 'com.netflix.nebula:nebula-publishing-plugin:13.6.1'
-        classpath 'com.palantir.baseline:gradle-baseline-java:2.3.0'
+        classpath 'com.palantir.baseline:gradle-baseline-java:2.13.0'
         classpath 'com.palantir.gradle.gitversion:gradle-git-version:0.12.2'
         classpath 'com.palantir.gradle.consistentversions:gradle-consistent-versions:1.12.4'
         classpath 'gradle.plugin.org.inferred:gradle-processors:3.1.0'
diff --git a/gradle-conjure/src/main/java/com/palantir/gradle/conjure/CompileConjurePythonTask.java b/gradle-conjure/src/main/java/com/palantir/gradle/conjure/CompileConjurePythonTask.java
index e492cfeb1..ae6218f03 100644
--- a/gradle-conjure/src/main/java/com/palantir/gradle/conjure/CompileConjurePythonTask.java
+++ b/gradle-conjure/src/main/java/com/palantir/gradle/conjure/CompileConjurePythonTask.java
@@ -31,7 +31,7 @@
 public class CompileConjurePythonTask extends ConjureGeneratorTask {
 
     @Override
-    protected final Map<String, Supplier<Object>> requiredOptions(File file) {
+    protected final Map<String, Supplier<Object>> requiredOptions(File _file) {
         return ImmutableMap.of(
                 "packageName",
                 this::getProjectName,
diff --git a/gradle-conjure/src/main/java/com/palantir/gradle/conjure/CompileConjureTypeScriptTask.java b/gradle-conjure/src/main/java/com/palantir/gradle/conjure/CompileConjureTypeScriptTask.java
index ba4be8f35..04c03f60a 100644
--- a/gradle-conjure/src/main/java/com/palantir/gradle/conjure/CompileConjureTypeScriptTask.java
+++ b/gradle-conjure/src/main/java/com/palantir/gradle/conjure/CompileConjureTypeScriptTask.java
@@ -57,7 +57,7 @@ public void execute(Task task) {
     }
 
     @Override
-    protected final Map<String, Supplier<Object>> requiredOptions(File file) {
+    protected final Map<String, Supplier<Object>> requiredOptions(File _file) {
         return ImmutableMap.of(
                 "packageName",
                 this::getPackageName,
diff --git a/gradle-conjure/src/main/java/com/palantir/gradle/conjure/ConjureGeneratorTask.java b/gradle-conjure/src/main/java/com/palantir/gradle/conjure/ConjureGeneratorTask.java
index fde596f58..dd3445a7b 100644
--- a/gradle-conjure/src/main/java/com/palantir/gradle/conjure/ConjureGeneratorTask.java
+++ b/gradle-conjure/src/main/java/com/palantir/gradle/conjure/ConjureGeneratorTask.java
@@ -45,7 +45,7 @@ public ConjureGeneratorTask() {
         // main task. Intentionally not using a lambda because this breaks Gradle caching
         doLast(new Action<Task>() {
             @Override
-            public void execute(Task task) {
+            public void execute(Task _task) {
                 compileFiles();
             }
         });
@@ -83,7 +83,7 @@ public final GeneratorOptions getOptions() {
      * Where to put the output for the given input source file.
      * This should return a directory that's under {@link #getOutputDirectory()}.
      */
-    protected File outputDirectoryFor(File file) {
+    protected File outputDirectoryFor(File _file) {
         return getOutputDirectory();
     }
 
@@ -116,7 +116,7 @@ public void compileFiles() {
      * What options are required, along with suppliers for obtaining their default values if they were not defined in
      * the {@link #getOptions() options}.
      */
-    protected Map<String, Supplier<Object>> requiredOptions(File file) {
+    protected Map<String, Supplier<Object>> requiredOptions(File _file) {
         return ImmutableMap.of();
     }
 }
diff --git a/gradle-conjure/src/main/java/com/palantir/gradle/conjure/ConjurePlugin.java b/gradle-conjure/src/main/java/com/palantir/gradle/conjure/ConjurePlugin.java
index ba4867250..8e8acee63 100644
--- a/gradle-conjure/src/main/java/com/palantir/gradle/conjure/ConjurePlugin.java
+++ b/gradle-conjure/src/main/java/com/palantir/gradle/conjure/ConjurePlugin.java
@@ -693,7 +693,7 @@ private static Copy getConjureSources(Project project) {
         // (see https://github.com/gradle/gradle/issues/5510)
         copyConjureSourcesTask.doFirst(new Action<Task>() {
             @Override
-            public void execute(Task task) {
+            public void execute(Task _task) {
                 GFileUtils.deleteDirectory(buildDir);
             }
         });
diff --git a/gradle-conjure/src/main/java/com/palantir/gradle/conjure/ExtractExecutableTask.java b/gradle-conjure/src/main/java/com/palantir/gradle/conjure/ExtractExecutableTask.java
index 57880640a..c463cb728 100644
--- a/gradle-conjure/src/main/java/com/palantir/gradle/conjure/ExtractExecutableTask.java
+++ b/gradle-conjure/src/main/java/com/palantir/gradle/conjure/ExtractExecutableTask.java
@@ -56,7 +56,7 @@ public ExtractExecutableTask() {
 
         doFirst(new Action<Task>() {
             @Override
-            public void execute(Task task) {
+            public void execute(Task _task) {
                 Set<String> rootDirectories = new HashSet<>();
                 getProject().tarTree(tarFile.get()).visit(new FileVisitor() {
                     @Override
@@ -71,7 +71,7 @@ public void visitDir(FileVisitDetails dirDetails) {
                     }
 
                     @Override
-                    public void visitFile(FileVisitDetails fileDetails) {}
+                    public void visitFile(FileVisitDetails _fileDetails) {}
                 });
                 if (rootDirectories.size() != 1) {
                     throw new GradleException(String.format(
@@ -84,7 +84,7 @@ public void visitFile(FileVisitDetails fileDetails) {}
 
         doLast(new Action<Task>() {
             @Override
-            public void execute(Task task) {
+            public void execute(Task _task) {
                 getLogger().info("Extracted into {}", getOutputDirectory());
                 // Ensure the executable exists
                 Preconditions.checkState(