Ryan Marcotte
Ryan Marcotte
I'll note that the injection of `DependencyMetadata` in a decorator does work and behaves as expected in the following scenario: ``` csharp public class DecoratorWithDependencyMetadataTests { [Fact] public void SuccessfullyAppliesDecoratorWithDependencyMetadataToType()...
For sure. I will use `DecoratorContext` for my particular scenario as you suggested in #878 . Thanks again!
Instead of adding support for `DependencyMetadata` in decorators... Might it be easier / make more sense to throw an exception that directs people to use `DecoratorContext` instead of `DependencyMetadata`? My...
I'd be happy to contribute a pull request once the desired behavior is finalized. Let me know!
Awesome! I can add some documentation to the `README`. Would you prefer that be done as part of the PR or after a new version of the package is released?
Documentation added to the existing PR
The following would work ``` csharp Result.Try(() => DoSomething(), exception => MapFailure(exception)) .Do(_ => Finally(), _ => Finally()) ```
I debated whether the `ApplyOnFailure` method should be for all `Result` types or just `Result`... went with the latter because we marked `Apply(Action)` as obsolete and steer developers toward using...
For a bit of background, `ApplyOnFailure` is an extension method I ended up writing for a couple client projects that am now just proposing for the library. >The version of...
>If I recall, the reasons for explicitly handling both cases for Apply were: > >- For clarity, to make it obvious that there are two cases which could occur and...