Stéphane,
I have created six PRs for each of my commits, as detailed below. However I recommend that you consider 231, 232, 233 as a single patch as they are interrelated. You should also assess the impact to users of reactivemongo as it removes the ability to supply RM with an external ActorSystem and instead always creates on for RM. If that "feature" can't be done away with, there may be other things we can do, but I'd need to discuss them with you. The goal of those patches is to make it possible to have multiple MongoDrivers that keep their business separate and other enhancements to make startup/shutdown much more reliable. The other patches, however, are straight up improvements that you will probably want. Here's the list:
- Reliable Shutdown, Non-opinionated Logging, Self-Contained Actor System - #231
- Make Monitor, Connection and Supervisor names unique - #232
- Permit MongoDriver to be configured - #233
- Reduce Warnings And Deprecations - #234
- Upgrade to Scala 2.11.4 - #235
- Fix Equality And Other Issues - #236
Let me know how you would like to proceed.
Best,
Reid.
On Monday, December 22, 2014 8:12:29 PM UTC-5, Stéphane Godbillon wrote:
I've had a glance to your commits and your changes seem to be quite interesting! :) It would be great to issue a PR for them. Then I will review it closely. If you're willing to do so, please do not squash – it's better to keep the commits meaningful.
Cheers,