search

invenia / Memento.jl

A flexible logging library for Julia
https://invenia.github.io/Memento.jl/latest
Other
88 stars 14 forks source link

Default level to info #70

Closed rofinn closed 6 years ago

rofinn commented 6 years ago

There has been some discussion around changing the default logging level to "info" to better align with base. I usually don't want to see "info" messages in production code, but I can always do Memento.config("warn") if necessary. Please up or down vote this issue to express your preference (upvote if you'd like the default to be "info").

omus commented 6 years ago

I would be ok with it. Maybe info should be the default when running interactive otherwise stick to warn?

rofinn commented 6 years ago

I was thinking about that, but I'd prefer if we had consistent behaviour between interactive and noninteractive.

omus commented 6 years ago

I can agree with having the behaviour be consistent is important. What about mentioning in the documentation to have Memento.config("info") as part of the ~/.juliarc.jl file?

rofinn commented 6 years ago

Running Memento.config("info") isn't the same thing. We'd want to add a recursive flag for that to work... which we might want to do regardless.

rofinn commented 6 years ago

I'll try testing a few permutations with several packages to see what works best and report back. If we can manage most use cases with just a single function call then I'll be happy.