Catch broader FileConfigError rather than FileDecodeError in is_used

nathanjmcdougall · nathanjmcdougall · commit 32d0ac497545 · 2025-06-28T13:07:08.000+12:00
This won't make any material difference, but makes more sense conceptually.
diff --git a/src/usethis/_tool/base.py b/src/usethis/_tool/base.py
@@ -29,7 +29,7 @@
 from usethis._tool.config import ConfigSpec, NoConfigValue
 from usethis._tool.pre_commit import PreCommitConfig
 from usethis._tool.rule import RuleConfig
-from usethis.errors import FileDecodeError
+from usethis.errors import FileConfigError
 
 if TYPE_CHECKING:
     from pathlib import Path
@@ -109,7 +109,7 @@ def is_used(self) -> bool:
         2. Whether any of the tool's managed files are in the project.
         3. Whether any of the tool's managed config file sections are present.
         """
-        decode_err_by_name: dict[str, FileDecodeError] = {}
+        decode_err_by_name: dict[str, FileConfigError] = {}
         _is_used = False
 
         _is_used = any(
@@ -119,13 +119,13 @@ def is_used(self) -> bool:
         if not _is_used:
             try:
                 _is_used = self.is_declared_as_dep()
-            except FileDecodeError as err:
+            except FileConfigError as err:
                 decode_err_by_name[err.name] = err
 
         if not _is_used:
             try:
                 _is_used = self.is_config_present()
-            except FileDecodeError as err:
+            except FileConfigError as err:
                 decode_err_by_name[err.name] = err
 
         for name, decode_err in decode_err_by_name.items():
