Skip to content

Commit

Permalink
Remove unnecessary possible_modules config option
Browse files Browse the repository at this point in the history
  • Loading branch information
xylix committed Mar 5, 2020
1 parent b14b1be commit 9e4b15e
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 11 deletions.
5 changes: 0 additions & 5 deletions aw_qt/config.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,10 +5,6 @@
import json

default_settings = {
"possible_modules": json.dumps(["aw-server",
"aw-server-rust",
"aw-watcher-afk",
"aw-watcher-window", ]),
"autostart_modules": json.dumps(["aw-server-rust",
"aw-server",
"aw-watcher-afk",
Expand All @@ -26,5 +22,4 @@ class AwQtSettings:
def __init__(self, testing: bool):
config_section = qt_config["aw-qt" if not testing else "aw-qt-testing"]

self.possible_modules: List[str] = json.loads(config_section["possible_modules"])
self.autostart_modules: List[str] = json.loads(config_section["autostart_modules"])
6 changes: 0 additions & 6 deletions aw_qt/manager.py
Original file line number Diff line number Diff line change
Expand Up @@ -182,12 +182,6 @@ def __init__(self, testing: bool = False) -> None:
self.autostart_modules: Set[str] = set(self.settings.autostart_modules)
self.testing = testing

for name in self.settings.possible_modules:
if _locate_executable(name):
self.modules[name] = Module(name, testing=testing)
else:
logger.warning("Module '{}' not found but was in possible modules".format(name))
# Is this actually a good way to do this? merged from dev/autodetect-modules
self.discover_modules()

def discover_modules(self) -> None:
Expand Down

0 comments on commit 9e4b15e

Please sign in to comment.