Required

[madkins23]

I went through the requirements in the slog.Handler documentation and the tests in slogtest and added some testing to your console handler tests. This is how I found the zerotime issue that you just pushed.

Everything else seems fine. Half of what I did was to add comments to tests detailing why they are important.

I'm not sure you'll want these additions, but you're welcome to them. The only reason I can think of to add them is the chance that the stuff I did add might catch some future error.

I won't be offended if you just ignore this. 😉

BTW, thanks for doing this!

[phsym]

That's very cool, thanks a lot for this. I'm running out of time for today, but I'll review very soon. There are high chances that I'll merge this. There's no reason to accept more tests