Ignore ill-formed INI files instead of crashing

Author: DimitriPapadopoulos

codespell_lib/_codespell.py

@@ -118,6 +118,7 @@
 EX_OK = 0
 EX_USAGE = 64
 EX_DATAERR = 65
+EX_CONFIG = 78
 
 # OPTIONS:
 #
@@ -586,7 +587,7 @@ def parse_options(
             used_cfg_files.append(cfg_file)
 
     # Use config files
-    config.read(cfg_files)
+    config.read(used_cfg_files)
     if config.has_section("codespell"):
         # Build a "fake" argv list using option name and value.
         cfg_args = []
@@ -1021,7 +1022,14 @@ def _script_main() -> int:
 
 def main(*args: str) -> int:
     """Contains flow control"""
-    options, parser, used_cfg_files = parse_options(args)
+    try:
+        options, parser, used_cfg_files = parse_options(args)
+    except configparser.Error as e:
+        print(
+            f"ERROR: ill-formed config file: {e.message}",
+            file=sys.stderr,
+        )
+        return EX_CONFIG
 
     # Report used config files
     if not options.quiet_level & QuietLevels.CONFIG_FILES:

codespell_lib/tests/test_basic.py

@@ -13,7 +13,13 @@
 import pytest
 
 import codespell_lib as cs_
-from codespell_lib._codespell import EX_DATAERR, EX_OK, EX_USAGE, EX_CONFIG, uri_regex_def
+from codespell_lib._codespell import (
+    EX_DATAERR,
+    EX_OK,
+    EX_USAGE,
+    EX_CONFIG,
+    uri_regex_def,
+)
 
 
 def test_constants() -> None:
@@ -1076,7 +1082,7 @@ def test_ill_formed_ini_config_file(
     assert isinstance(result, tuple)
     code, _, stderr = result
     assert code == 78
-    assert("ill-formed config file" in stderr)
+    assert "ill-formed config file" in stderr
 
 
 @pytest.mark.parametrize("kind", ("toml", "cfg"))