Move the pulse message receiver into its own object, in pulselistener.lib, dedicated to listening to one queue+route, and that simply enqueues the received messages in the bus.
Remove the Hook & PulseHook base classes
HookPhabricator did not use any pulse message, so not much change there
HookCodeCoverage now has an explicit workflow, listening to the internal pulse queue, and triggering new tasks as before.
This PR does several things:
pulselistener.lib, dedicated to listening to one queue+route, and that simply enqueues the received messages in the bus.Hook&PulseHookbase classesHookPhabricatordid not use any pulse message, so not much change thereHookCodeCoveragenow has an explicit workflow, listening to the internal pulse queue, and triggering new tasks as before.