nblumhardt / serilog-timings

Extends Serilog with support for timed operations
Apache License 2.0
218 stars 22 forks source link

Support passing multiple enrichers to Operation.Complete #8

Closed ben-foster-cko closed 5 years ago

ben-foster-cko commented 8 years ago

Currently, we can only pass one event log property to Operation.Complete. We have the need to pass multiple properties - we don't want to destructure our result object.

Could we instead pass an array of enrichers to Operation.Complete to "enrich" the operation result?

benfoster commented 8 years ago

Happy to send a PR if you're happy for me to add the following overload:

Complete(IEnumerable<ILogEventEnricher> resultEnrichers)
nblumhardt commented 5 years ago

Completed in #30