From 46413c2c52cd2325bd207910f5dad81ba10d7bdc Mon Sep 17 00:00:00 2001 From: jack1142 <6032823+jack1142@users.noreply.github.com> Date: Tue, 23 Apr 2019 02:07:52 +0200 Subject: [PATCH] [V3 Downloader] Stop including subpackages in cog list (#2590) Before this change, `Repo.available_modules` would also contain subpackages, which is unintended --- redbot/cogs/downloader/repo_manager.py | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/redbot/cogs/downloader/repo_manager.py b/redbot/cogs/downloader/repo_manager.py index 80581308c..a12ec9030 100644 --- a/redbot/cogs/downloader/repo_manager.py +++ b/redbot/cogs/downloader/repo_manager.py @@ -173,9 +173,7 @@ class Repo(RepoJSONMixin): Installable(location=name) ) """ - for file_finder, name, is_pkg in pkgutil.walk_packages( - path=[str(self.folder_path)], onerror=lambda name: None - ): + for file_finder, name, is_pkg in pkgutil.iter_modules(path=[str(self.folder_path)]): if is_pkg: curr_modules.append(Installable(location=self.folder_path / name)) self.available_modules = curr_modules