Disable UnusedVariable in favour of StrictUnusedVariable (#888)

Disable `UnusedVariable` error prone rule by default
palantir · Sep 24, 2019 · a1d3c42 · a1d3c42
1 parent 95b78c6
commit a1d3c42
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/changelog/@unreleased/pr-888.v2.yml b/changelog/@unreleased/pr-888.v2.yml
@@ -0,0 +1,5 @@
+type: improvement
+improvement:
+  description: Disable `UnusedVariable` error prone rule by default
+  links:
+  - https://github.com/palantir/gradle-baseline/pull/888
diff --git a/gradle-baseline-java/src/main/groovy/com/palantir/baseline/plugins/BaselineErrorProne.java b/gradle-baseline-java/src/main/groovy/com/palantir/baseline/plugins/BaselineErrorProne.java
@@ -180,6 +180,7 @@ private static void configureErrorProneOptions(
         errorProneOptions.setDisableWarningsInGeneratedCode(true);
         errorProneOptions.setExcludedPaths(
                 String.format("%s/(build|src/generated.*)/.*", project.getProjectDir().getPath()));
+        errorProneOptions.check("UnusedVariable", CheckSeverity.OFF);
         errorProneOptions.check("EqualsHashCode", CheckSeverity.ERROR);
         errorProneOptions.check("EqualsIncompatibleType", CheckSeverity.ERROR);
         errorProneOptions.check("StreamResourceLeak", CheckSeverity.ERROR);