smbache / loggr

Easy and flexible logging for R
Other
79 stars 6 forks source link

Rename deactivate_log() to something internally consistent #32

Open beansrowning opened 6 years ago

beansrowning commented 6 years ago

Maybe there is a greater reason for this that I am blind to, but I think it's a bit odd that all the other functions in the package are prefixed with log_, only to have the function to quit logging break this syntax.

Would log_close() or log_end() not work and be much more logical given the existing functions?
Just a thought.