Revert "Include more checks to be patched (#751)" (#757)

Revert "Include more checks to be patched (#751)"
palantir · Aug 19, 2019 · 441544f · 441544f
1 parent 58830cc
commit 441544f
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/changelog/@unreleased/revert-pr-751.v2.yml b/changelog/@unreleased/revert-pr-751.v2.yml
@@ -0,0 +1,5 @@
+type: fix
+fix:
+  description: Do not apply the suggested fixes for `UnusedMethod` and `UnusedVariable` which automaticall remove code with side effects.
+  links:
+  - https://github.com/palantir/gradle-baseline/pull/757
diff --git a/...ne-java/src/main/groovy/com/palantir/baseline/extensions/BaselineErrorProneExtension.java b/...ne-java/src/main/groovy/com/palantir/baseline/extensions/BaselineErrorProneExtension.java
@@ -33,9 +33,7 @@ public class BaselineErrorProneExtension {
 
             // Built-in checks
             "ArrayEquals",
-            "MissingOverride",
-            "UnusedMethod",
-            "UnusedVariable");
+            "MissingOverride");
 
     private final ListProperty<String> patchChecks;