Apply suggestions from code review

lsst · Mar 24, 2023 · 1365f07 · 1365f07
1 parent 2480e81
commit 1365f07
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/python/lsst/daf/butler/core/config.py b/python/lsst/daf/butler/core/config.py
@@ -468,7 +468,7 @@ def _processExplicitIncludes(self) -> None:
                             # Remote resource check might be expensive
                             if specific.exists():
                                 found = specific
-                                # TODO: do we need `break` here?
+                                break
                     if not found:
                         raise RuntimeError(f"Unable to find referenced include file: {fileName}")
 
@@ -543,7 +543,7 @@ def _splitIntoKeys(key: _ConfigKey) -> list[str | int]:
             # Copy the list to keep mypy quiet.
             return list(hierarchy)
         elif isinstance(key, Iterable):
-            return [k for k in key]
+            return list(key)
         else:
             # Do not try to guess.
             raise TypeError(f"Provided key [{key}] neither str nor iterable.")