Workrave flushing to log file too conservatively?

[jjramsey]

I notice that the main log file workrave.log in ~/.cache/workrave is often empty or maybe a day old, and it seems to me that this is fixed by changing what is currently line 67 of workrave/ui/app/main.cc from

logger->flush_on(spdlog::level::critical);

to

logger->flush_on(spdlog::level::info);

Of course, that means that Workrave would be writing to the log file more frequently, but the log file still doesn't seem to be written to that frequently, and the size of the file is still only a few kilobytes.

Maybe it might make more sense to flush on some level a bit less trivial than info but less severe than critical, like spdlog::level::warn?