Skip to content

Commit cf3850b

Browse files
committed
Address Copilot comments
1 parent eeb08f2 commit cf3850b

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

metaflow/extension_support/__init__.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -412,7 +412,7 @@ def _get_extension_packages(ignore_info_file=False, restrict_to_directories=None
412412

413413
# Temporary variables to support the loop below and make sure we loop through all
414414
# the paths in the submodule_search_locations including calling the path hooks.
415-
# We coudl skip calling things on the path hooks since the module was just imported
415+
# We could skip calling things on the path hooks since the module was just imported
416416
# by importlib so the values are probably already in submodule_search_locations but
417417
# there may be cases where we need to call multiple times. This also allows us to tie
418418
# the finders (ie: the path hooks) back to the distribution since they share a name.
@@ -544,7 +544,7 @@ def process_file(state: Dict[str, Any], root_dir: str, file: str):
544544
"Package '%s' defines more than one meta configuration: "
545545
"'%s' and '%s' (at least)"
546546
% (
547-
dist_name,
547+
state["name"],
548548
state["meta_module"],
549549
potential_meta_module,
550550
)
@@ -968,8 +968,9 @@ def _filter_files_package(pkg):
968968
# - if include_suffixes, only include those suffixes
969969
# - if *not* include_suffixes but exclude_suffixes, include everything *except*
970970
# files ending with that suffix
971+
new_files, new_full_path_files = [], []
972+
971973
if filter_function:
972-
new_files, new_full_path_files = [], []
973974
for short_file, full_file in zip(pkg["files"], pkg["full_path_files"]):
974975
try:
975976
if filter_function(os.path.join(EXT_PKG, short_file)):

0 commit comments

Comments
 (0)