From 18833c5e368a42893ec864f7737159087401f2aa Mon Sep 17 00:00:00 2001 From: Athanasius <Athanasius@miggy.org> Date: Mon, 26 Dec 2022 19:48:38 +0000 Subject: [PATCH] plugins: Comment *why* the "load plugins that are packages" first is there It's *NOT* that it will actually load the plugin *as* a package, as the code explicitly only loads `load.py` still. --- plug.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/plug.py b/plug.py index e0c69a44..92777acf 100644 --- a/plug.py +++ b/plug.py @@ -160,7 +160,10 @@ def load_plugins(master: tk.Tk) -> None: # noqa: CCR001 sys.path.append(config.plugin_dir) found = [] - # Load any plugins that are also packages first + # Load any plugins that are also packages first, but note it's *still* + # 100% relying on there being a `load.py`, as only that will be loaded. + # The intent here is to e.g. have EDMC-Overlay load before any plugins + # that depend on it. for name in sorted( os.listdir(config.plugin_dir_path), key=lambda n: (not os.path.isfile(os.path.join(config.plugin_dir_path, n, '__init__.py')), n.lower())