Closed cgallred closed 5 years ago
It looks like the commit in this PR got picked up by #14, is this PR still needed?
It looks like the commit in this PR got picked up by #14, is this PR still needed?
The decoupling of notification callbacks and test mode got picked up, because adding the 'deny deletes' option depended on it. But switching the callback logging to Serilog is only in this PR.
It looks like the commit in this PR got picked up by #14, is this PR still needed?
The decoupling of notification callbacks and test mode got picked up, because adding the 'deny deletes' option depended on it. But switching the callback logging to Serilog is only in this PR.
Or maybe not. Looks like I have a bad merge somewhere. I need to fix the notification callbacks.