Fix: Exit if any config error

python · Aug 15, 2023 · 7e3f281 · 7e3f281
1 parent 14418bc
commit 7e3f281
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/mypy/config_parser.py b/mypy/config_parser.py
@@ -471,7 +471,8 @@ def parse_section(
                 elif key == "strict":
                     pass  # Special handling below
                 else:
-                    print(f"{prefix}Unrecognized option: {key} = {section[key]}", file=stderr)
+                    msg=f"{prefix}Unrecognized option: {key} = {section[key]}"
+                    raise argparse.ArgumentTypeError(msg)
                 if invert:
                     dv = getattr(template, options_key, None)
                 else:
@@ -493,7 +494,8 @@ def parse_section(
                 try:
                     v = ct(section.get(key))
                 except argparse.ArgumentTypeError as err:
-                    print(f"{prefix}{key}: {err}", file=stderr)
+                    msg=f"{prefix}{key}: {err}"
+                    raise argparse.ArgumentTypeError(msg)
                     continue
             else:
                 print(f"{prefix}Don't know what type {key} should have", file=stderr)

diff --git a/mypy/main.py b/mypy/main.py
@@ -1249,7 +1249,11 @@ def set_strict_flags() -> None:
             setattr(options, dest, value)
 
     # Parse config file first, so command line can override.
-    parse_config_file(options, set_strict_flags, config_file, stdout, stderr)
+    try:
+        parse_config_file(options, set_strict_flags, config_file, stdout, stderr)
+    except argparse.ArgumentTypeError as err:
+        stderr.write(f"{err}")
+        sys.exit(1)
 
     # Set strict flags before parsing (if strict mode enabled), so other command
     # line options can override.