Rolf Kristensen
Rolf Kristensen
Must admit I like using default-parameters, so do not fully agree with Sonar (I like that one can see what default-value that will be used when not provided). But anyway...
Closing pull-request is it has unresolved conflicts, and it also introduces breaking changes that must wait for NLog 6.0
What should happen on LoggingConfiguration-Reload (CloseTarget / InitializeTarget) ? The header-logic is mostly for the CSV-layout. Why not manually write a log-message at application-startup like "Starting..." ? Alternative solutions could...
@d3ns1ty Make sure to read https://github.com/NLog/NLog/issues/2119#issuecomment-302490558 before you start implementing a solution.
Especially when a new instance of the target is created on reload :)
As long it is done in a target wrapper and not polluting the file-target even more with random logic then it is fine with me
Anything where you need a starting and stopping event.
But remember a good library doesn't try to do everything. But should be good at its core stuff while being open and extensible. Careful with not adding everything but the...
@mwagenfuehr Think you are very welcome to create a PullRequest and implement this feature.
Currently we have methods like this one: `public void Trace(string message, TArgument argument)` One could add an extra version with: ``` [CLSCompliant(false)] public void Trace(string message, TArgument argument) where TArgument...