Remove unused run_before_tasks

Author: linuxdaemon

CHANGELOG.md

@@ -20,6 +20,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Apply rate limiting to regex hooks
 - Ensure event order is deterministic
 ### Fixed
+- Ensure event order is deterministic
 - Fix matching exception in horoscope test
 - Fix youtube.py ISO time parse
 - Fix grammatical error in food sentence (beer)
@@ -34,6 +35,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - metacritic.py removed due to broken scraper and lack of maintainer
 - amazon.py removed due to broken scraper and no maintainer
 - newegg.py removed due to broken scraper and no maintainer
+- Remove unused run_before events/tasks
 
 ## [1.3.0] 2020-03-17
 ### Added

cloudbot/bot.py

@@ -311,11 +311,10 @@ async def process(self, event):
         """
         :type event: Event
         """
-        run_before_tasks = []
         tasks = []
         halted = False
 
-        def add_hook(hook, _event, _run_before=False):
+        def add_hook(hook, _event):
             nonlocal halted
             if halted:
                 return False
@@ -324,10 +323,7 @@ def add_hook(hook, _event, _run_before=False):
                 return True
 
             coro = self.plugin_manager.launch(hook, _event)
-            if _run_before:
-                run_before_tasks.append(coro)
-            else:
-                tasks.append(coro)
+            tasks.append(coro)
 
             if hook.action is Action.HALTALL:
                 halted = True
@@ -340,10 +336,8 @@ def add_hook(hook, _event, _run_before=False):
 
         # Raw IRC hook
         for raw_hook in self.plugin_manager.catch_all_triggers:
-            # run catch-all coroutine hooks before all others - TODO: Make this a plugin argument
-            run_before = not raw_hook.threaded
             if not add_hook(
-                raw_hook, Event(hook=raw_hook, base_event=event), _run_before=run_before
+                raw_hook, Event(hook=raw_hook, base_event=event)
             ):
                 # The hook has an action of Action.HALT* so stop adding new tasks
                 break
@@ -424,5 +418,4 @@ def add_hook(hook, _event, _run_before=False):
                         break
 
         # Run the tasks
-        await asyncio.gather(*run_before_tasks, loop=self.loop)
         await asyncio.gather(*tasks, loop=self.loop)