search

phuslu / log

Fastest structured logging
MIT License
672 stars 44 forks source link

FileWriter comment mentions MaxAge but that field does not exist #56

Closed cgrinds closed 2 years ago

cgrinds commented 2 years ago

The comment for FileWriter mentions MaxAge but that field does not exist.

Any files with an encoded timestamp older than MaxAge days are deleted, regardless of MaxBackups

MaxAge would be nice to have if this library is meant to be a drop-in replacement for Lumberjack.

phuslu commented 2 years ago

Oh, I removed the comments now. I think "MaxAge" is a wrong design for log libs, I recommend that way https://github.com/phuslu/log#rotating-file-writer