-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
FileTarget: Fix archiving for ArchiveFileName without a pattern. #1109
Conversation
Current coverage is
|
@@ -524,6 +538,23 @@ protected internal string NewLineChars | |||
} | |||
} | |||
|
|||
private void RefreshFileArchive() | |||
{ | |||
var dummyEvent = new LogEventInfo(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We have LogEventInfo.CreateNullEvent()
for this :)
hey @bhaeussermann, thanks for your contribution again! I just checked your code in a short time (not done yet), and I have added some code comments. We will review you code more in depth soon. We are really pleased with your contributions! Nice findings and fixes! Also happy with your good explanation :) |
{ | ||
logger.Debug("ccc"); | ||
} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here I extended the test to check that the second archive is created.
Sure! :P |
Refactor - use LogEventInfo.CreateNullEvent().
Cool! Check your mail and https://gitter.im/NLog/devs |
FileTarget: Fix archiving for ArchiveFileName without a pattern.
Thanks! I has been merged! |
Fixes issue #1108