diff --git a/refactorings/src/main/java/com/google/common/truth/refactorings/CorrespondenceSubclassToFactoryCall.java b/refactorings/src/main/java/com/google/common/truth/refactorings/CorrespondenceSubclassToFactoryCall.java index c360c6268..a8a265b7d 100644 --- a/refactorings/src/main/java/com/google/common/truth/refactorings/CorrespondenceSubclassToFactoryCall.java +++ b/refactorings/src/main/java/com/google/common/truth/refactorings/CorrespondenceSubclassToFactoryCall.java @@ -27,7 +27,6 @@ import static com.google.common.truth.refactorings.CorrespondenceSubclassToFactoryCall.MemberType.COMPARE_METHOD; import static com.google.common.truth.refactorings.CorrespondenceSubclassToFactoryCall.MemberType.CONSTRUCTOR; import static com.google.common.truth.refactorings.CorrespondenceSubclassToFactoryCall.MemberType.TO_STRING_METHOD; -import static com.google.errorprone.BugPattern.ProvidesFix.REQUIRES_HUMAN_ATTENTION; import static com.google.errorprone.BugPattern.SeverityLevel.SUGGESTION; import static com.google.errorprone.fixes.SuggestedFixes.compilesWithFix; import static com.google.errorprone.matchers.Description.NO_MATCH; @@ -92,8 +91,7 @@ @BugPattern( name = "CorrespondenceSubclassToFactoryCall", summary = "Use the factory methods on Correspondence instead of defining a subclass.", - severity = SUGGESTION, - providesFix = REQUIRES_HUMAN_ATTENTION) + severity = SUGGESTION) public final class CorrespondenceSubclassToFactoryCall extends BugChecker implements ClassTreeMatcher { @Override diff --git a/refactorings/src/main/java/com/google/common/truth/refactorings/FailWithFacts.java b/refactorings/src/main/java/com/google/common/truth/refactorings/FailWithFacts.java index 5978504ee..d47b35cb4 100644 --- a/refactorings/src/main/java/com/google/common/truth/refactorings/FailWithFacts.java +++ b/refactorings/src/main/java/com/google/common/truth/refactorings/FailWithFacts.java @@ -18,7 +18,6 @@ import static com.google.common.base.CharMatcher.inRange; import static com.google.common.base.CharMatcher.whitespace; -import static com.google.errorprone.BugPattern.ProvidesFix.REQUIRES_HUMAN_ATTENTION; import static com.google.errorprone.BugPattern.SeverityLevel.SUGGESTION; import static com.google.errorprone.matchers.Description.NO_MATCH; import static com.google.errorprone.matchers.Matchers.instanceMethod; @@ -57,8 +56,7 @@ @BugPattern( name = "FailWithFacts", summary = "Use the new key-value-style failure API instead of the deprecated one.", - severity = SUGGESTION, - providesFix = REQUIRES_HUMAN_ATTENTION) + severity = SUGGESTION) public final class FailWithFacts extends BugChecker implements MethodInvocationTreeMatcher { @Override public Description matchMethodInvocation(MethodInvocationTree tree, VisitorState state) { diff --git a/refactorings/src/main/java/com/google/common/truth/refactorings/NamedToWithMessage.java b/refactorings/src/main/java/com/google/common/truth/refactorings/NamedToWithMessage.java index 0e9f4e137..411ce6ddc 100644 --- a/refactorings/src/main/java/com/google/common/truth/refactorings/NamedToWithMessage.java +++ b/refactorings/src/main/java/com/google/common/truth/refactorings/NamedToWithMessage.java @@ -18,7 +18,6 @@ import static com.google.common.collect.ImmutableSet.toImmutableSet; import static com.google.common.collect.Iterables.getOnlyElement; -import static com.google.errorprone.BugPattern.ProvidesFix.REQUIRES_HUMAN_ATTENTION; import static com.google.errorprone.BugPattern.SeverityLevel.SUGGESTION; import static com.google.errorprone.matchers.Description.NO_MATCH; import static com.google.errorprone.matchers.Matchers.anyOf; @@ -62,8 +61,7 @@ @BugPattern( name = "NamedToWithMessage", summary = "Use assertWithMessage(...)/withMessage(...) instead of the deprecated named(...).", - severity = SUGGESTION, - providesFix = REQUIRES_HUMAN_ATTENTION) + severity = SUGGESTION) public final class NamedToWithMessage extends BugChecker implements MethodInvocationTreeMatcher { @Override public Description matchMethodInvocation(MethodInvocationTree namedCall, VisitorState state) { diff --git a/refactorings/src/main/java/com/google/common/truth/refactorings/StoreActualValueInField.java b/refactorings/src/main/java/com/google/common/truth/refactorings/StoreActualValueInField.java index ec5ec30c4..ea2e7e21e 100644 --- a/refactorings/src/main/java/com/google/common/truth/refactorings/StoreActualValueInField.java +++ b/refactorings/src/main/java/com/google/common/truth/refactorings/StoreActualValueInField.java @@ -18,7 +18,6 @@ import static com.google.common.collect.ImmutableSet.toImmutableSet; import static com.google.common.collect.Iterables.getOnlyElement; -import static com.google.errorprone.BugPattern.ProvidesFix.REQUIRES_HUMAN_ATTENTION; import static com.google.errorprone.BugPattern.SeverityLevel.SUGGESTION; import static com.google.errorprone.fixes.SuggestedFix.replace; import static com.google.errorprone.matchers.Description.NO_MATCH; @@ -73,8 +72,7 @@ name = "StoreActualValueInField", summary = "Store the actual value locally instead of using the deprecated actual() and getSubject().", - severity = SUGGESTION, - providesFix = REQUIRES_HUMAN_ATTENTION) + severity = SUGGESTION) public final class StoreActualValueInField extends BugChecker implements MethodInvocationTreeMatcher { @Override