Closed nbrooke closed 4 years ago
Made the following changes since the last review:
I haven't bumped the version / triggered a release yet, since I'm not totally convinced it's not still going to need some other changes (either to support the stuff I'm doing in Spies, or to maybe address the affect on the project that is using the notification support). I'm just integrating directly from master on Spies, @JeremyChiang you should maybe try pulling the latest in at some point and see if you can get it working reasonably using the LogDestination instead of notification, if there are any changes that need to be made there.
Also includes some associated cleanup, including: