Remove the flag --fail_on_errors.

RELNOTES:none.
PiperOrigin-RevId: 197923013

1 files changed, 1 insertions, 16 deletions

diff --git a/src/java_tools/import_deps_checker/java/com/google/devtools/build/importdeps/Main.java b/src/java_tools/import_deps_checker/java/com/google/devtools/build/importdeps/Main.java
index 7c7c034c7e..73e71b5f1a 100644
--- a/src/java_tools/import_deps_checker/java/com/google/devtools/build/importdeps/Main.java
+++ b/src/java_tools/import_deps_checker/java/com/google/devtools/build/importdeps/Main.java
@@ -132,20 +132,9 @@ public class Main {
         help = "The rule label of the current target under analysis.")
     public String ruleLabel;
 
-    // TODO (cnsun): remove this flag.
-    @Option(
-      name = "fail_on_errors",
-      defaultValue = "false",
-      documentationCategory = OptionDocumentationCategory.UNDOCUMENTED,
-      effectTags = {OptionEffectTag.UNKNOWN},
-      deprecationWarning = "This will be replaced with --checking_mode.",
-      help = "Fail on incomplete dependencies, otherwise emit warnings."
-    )
-    public boolean failOnErrors;
-
     @Option(
       name = "checking_mode",
-      defaultValue = "null",
+      defaultValue = "WARNING",
       documentationCategory = OptionDocumentationCategory.UNDOCUMENTED,
       effectTags = {OptionEffectTag.UNKNOWN},
       converter = CheckingModeConverter.class,
@@ -228,10 +217,6 @@ public class Main {
         options.jdepsOutput == null || !Files.isDirectory(options.jdepsOutput),
         "Invalid value of --jdeps_output: '%s'",
         options.jdepsOutput);
-    if (options.checkingMode == null) {
-      // Convert --fail_on_errors to --checking_mode
-      options.checkingMode = options.failOnErrors ? CheckingMode.ERROR : CheckingMode.WARNING;
-    }
     if (!options.fullClasspath.containsAll(options.directClasspath)) {
       ArrayList<Path> missing = Lists.newArrayList(options.directClasspath);
       missing.removeAll(options.fullClasspath);