Warren

Results 7 comments of Warren

Great. Thanks for a great tool otherwise. I'll see if I get a chance to look myself at some point.

Ah. That makes sense. I think that seems correct then. What is missing is adding the new files to the staged list. That should be possible as an additional part...

> > Tbh I am also quite surprised that there is not even a response by the maintainers to this pull request. > > Only 6 months? [Those are rookie...

The `IAmAHandlerFactory` implementation needs to change to `IAmAHandlerFactorySync`. The documentation that's up seems to be out of date. I would also suggest an exception should be thrown on [this line](https://github.com/BrighterCommand/Brighter/blob/master/src/Paramore.Brighter/CommandProcessor.cs#L120)...

@DevJonny ofcourse. Thanks for the swift response!

PR #12172 should resolve all of those, but no work has been done on this repo since September from the looks of it.