Skip to content
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 - Failing to delete old archive files, should not stop logging #1867

Merged
merged 1 commit into from
Dec 21, 2016

Conversation

snakefoot
Copy link
Contributor

@snakefoot snakefoot commented Dec 18, 2016

See also #1866

Have also changed the compression logic code from waiting forever to "only" 12 secs.

This change is Reviewable

@codecov-io
Copy link

codecov-io commented Dec 18, 2016

Current coverage is 82% (diff: 53%)

Merging #1867 into master will decrease coverage by <1%

@@             master      #1867   diff @@
==========================================
  Files           276        276          
  Lines         18503      18519    +16   
  Methods        2859       2860     +1   
  Messages          0          0          
  Branches       2137       2143     +6   
==========================================
  Hits          15094      15094          
- Misses         2917       2928    +11   
- Partials        492        497     +5   

Sunburst

Powered by Codecov. Last update cf507d5...0b4775e

@304NotModified
Copy link
Member

Thanks!

@304NotModified 304NotModified added file-target bug Bug report / Bug fix labels Dec 21, 2016
@304NotModified 304NotModified added this to the 4.4.1 milestone Dec 21, 2016
@304NotModified 304NotModified merged commit a7293bb into NLog:master Dec 21, 2016
@snakefoot snakefoot deleted the FileTargetMaxArchiveFiles branch October 10, 2017 20:39
@304NotModified 304NotModified added the file-archiving Issues with archiving with the file target label Aug 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug report / Bug fix file-archiving Issues with archiving with the file target file-target
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants