Closed aviks closed 11 years ago
@aviks, thanks for opening all of these issues and pull requests. Getting rid of LogLevel
is definitely worthwhile. I was just lazy when implementing this (the enum type is borrowed from one of @vtjnash's pull requests).
The loglevel in the logging line is displayed as
LogLevel(WARNING)
. This has 9 characters of redundant information. I think it would be better to display this simply asWARNING
. Or better still, justWARN