|
|
|
|
|
by ericyd
637 days ago
|
|
Looks like a cool project, and props for making it universal for all runtimes. A few pieces of feedback: 1. I don't understand the use case for emojis. If I'm using a logging library in production it's so I can pipe my structured logs into an observability tool. Do emojis provide value in this case? 2. Consider including timestamps by default and adding a config option to remove of needed. I personally can't think of a time where I wouldn't want timestamps in my logs. Lastly a question: Just because other loggers don't claim to be universal, do they actually break on Deno/Bun? |
|
As for emoji's, they are off by default and are really only for dev with the pretty logger. You are right in that you probably would not want them in production. You would probably use a different formatter like "standard" or "json" anyway which doesn't allow them.