Darrin W. Cullop

Results 24 comments of Darrin W. Cullop

Sorry but that PR does not address this. I was just being too chatty and mentioned what other issues I was considering addressing. My apologies. I will create a PR...

@JunkyXL86 Please note that `TransformManyAsync` and `TransformManySafeAsync` where implemented by #824. Thanks for the idea. Please let us know if you have any others.

Is this still an open issue? I implemented it for my current project and would be delighted for the chance to contribute as this library is amazing and has been...

I would be open to buffering up the changes, but I don't think we'll need that because, after further consideration, I think the OP had the right of it: the...

Okay, I will pursue implementing an overload for the MergeMany operator that works specifically on ChangeSets and use the implementation of DynamicCombiner (specifically the Or operations) as the starting point...

I used DynamicCombiner as a starting point, but since I only needed to handle the "Or" case, I could optimize for each of the operations instead of needing to have...

Please add some description to your PR (describe what you are changing and why). It doesn't have to be long, but it is for posterity.

> Looks like the build failures are complaining about calling LINQ methods on objects that now support direct calls (E.G. `.Count()` versus `.Count`), and only in `Tests` code. Another case...

Even though it's a breaking change, I think this is a good thing. Waiting until a major release mitigates the breaking change somewhat... Thanks for this. Maybe we should examine...

@RolandPheasant If this is what you had in mind, then accept it at your leisure. Or save it for later if you're not ready to make the breaking change. Later...