Added TODOs for future PRs

Author: linuxdaemon

cloudbot/clients/irc.py

@@ -126,6 +126,7 @@ def connect(self):
             self.bot.plugin_manager.launch(hook, Event(bot=self.bot, conn=self, hook=hook))
             for hook in self.bot.plugin_manager.connect_hooks
         ]
+        # TODO stop connecting if a connect hook fails?
         yield from asyncio.gather(*tasks)
 
     def quit(self, reason=None):

cloudbot/plugin.py

@@ -565,6 +565,7 @@ def __init__(self, filepath, filename, title, code):
         self.file_path = filepath
         self.file_name = filename
         self.title = title
+        # TODO clean up hook lists
         hooks = find_hooks(self, code)
         self.commands, self.regexes, self.raw_hooks, *hooks = hooks
         self.sieves, self.events, self.periodic, *hooks = hooks

plugins/cap.py

@@ -75,6 +75,7 @@ def on_cap(irc_paramlist, conn, bot, event):
         logger.info("New capabilities advertised: %s", irc_paramlist[-1])
         yield from handle_available_caps(conn, caplist, event, irc_paramlist, bot)
     elif subcmd == 'DEL':
+        # TODO add hooks for CAP removal
         logger.info("Capabilities removed by server: %s", irc_paramlist[-1])
         server_caps = conn.memory.setdefault('server_caps', {})
         for cap in caplist: