Better error message for invalid exclude types (#3764)

diff --git a/CHANGES.md b/CHANGES.md
index acb5a822674ba22db49ee5223ee641b40c07f973..bfa021617cbf7eb51a8c893dd241b288df43cf86 100644 (file)
--- a/CHANGES.md
+++ b/CHANGES.md
@@ -32,6 +32,8 @@
 - `.pytest_cache`, `.ruff_cache` and `.vscode` are now excluded by default (#3691)
 - Fix black not honouring `pyproject.toml` settings when running `--stdin-filename` and
   the `pyproject.toml` found isn't in the current working directory (#3719)
+- Black will now error if `exclude` and `extend-exclude` have invalid data types in
+  `pyproject.toml`, instead of silently doing the wrong thing (#3764)
 
 ### Packaging
 

diff --git a/src/black/__init__.py b/src/black/__init__.py
index 222cb3ca03d98ca059c3c09d270b97700558105a..b6611bef84b2e72048255f0b3e79ac2b29c31e80 100644 (file)
--- a/src/black/__init__.py
+++ b/src/black/__init__.py
@@ -157,6 +157,16 @@ def read_pyproject_toml(
             "target-version", "Config key target-version must be a list"
         )
 
+    exclude = config.get("exclude")
+    if exclude is not None and not isinstance(exclude, str):
+        raise click.BadOptionUsage("exclude", "Config key exclude must be a string")
+
+    extend_exclude = config.get("extend_exclude")
+    if extend_exclude is not None and not isinstance(extend_exclude, str):
+        raise click.BadOptionUsage(
+            "extend-exclude", "Config key extend-exclude must be a string"
+        )
+
     default_map: Dict[str, Any] = {}
     if ctx.default_map:
         default_map.update(ctx.default_map)