Nicholas Blumhardt

Results 455 comments of Nicholas Blumhardt

Thanks @mc0re ! It does look as if the one check in https://github.com/serilog/serilog/blob/dev/src/Serilog/Core/Logger.cs#L199 is superfluous. The comment is correct for the other (generic) overloads, but the `NoProperties` case doesn't need...

Hi! Thanks for the message. Unfortunately I can't see any way to implement this in the short-term; if you're using this in one project only, copying in/forking the relevant console...

Thanks for the suggestion, will need to loop back around and give it some thought :+1:

Thanks @agocke! To get the build green with these changes, the worker image in AppVeyor.yml will need to be updated to VS2022 (IIRC).

`dev` now includes a .NET 6 target so fingers crossed this will all come together 🤞

Thanks for the feedback, agreed on all points :-) Regarding the packaging, this does intersect with one other Serilog "v3" goal I've been giving some thought to. The fine-grained packaging...

_Platform_ was the best that I could come up with, but you're right that it's really an app model thing. _Serilog.AppModel.*_ doesn't roll off the tongue so well, though :-p...

`Bundle` makes me think of CSS/JavaScript assets :-D > The part that makes me a bit uncomfortable is turning Serilog into a metapackage I have essentially the same reservations -...

Thanks for the report, we'll dig in 👍

Hi @DarkMio - thanks for reaching out. Starting work on a proper fix for this now via https://github.com/serilog/serilog/pull/1750.