search

nblumhardt / serilog-timings

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

write log at begin #44

Closed mma42300 closed 2 years ago

mma42300 commented 2 years ago

Proposing a solution for the issue https://stackoverflow.com/questions/71873330/logging-the-begin-event-in-serilog-timings

nblumhardt commented 2 years ago

Thanks for the note and PR! My goal is to keep SerilogTimings fairly minimal, so I don't think I want to make this change right now, but it's a useful data point to have for when/if the topic comes up again 👍

Added a suggested alternative to you SO question.

mma42300 commented 2 years ago

Thanks for the note and PR! My goal is to keep SerilogTimings fairly minimal, so I don't think I want to make this change right now, but it's a useful data point to have for when/if the topic comes up again 👍

Added a suggested alternative to you SO question.

Thanks, your solution works perfectly, so my proposed modification is really unnecessary!

nblumhardt commented 2 years ago

Thanks 👍