Matteo Collina
Matteo Collina
I would prefer an example that does not depend on Chalk, but overall it's a good feature to have.
Not really, I would put it in a separate file for the ease of testing.
https://github.com/pinojs/pino-pretty/blob/e5e3f5473d76844615e17d67a1e9b55bb20d6385/lib/utils.js#L346 seems a good function to start with.
If we need to fork I prefer it to live in the pino org: anyway, good work! Could you start integrating that here?
You should be able to transfer the repo to this org. I'll handle the publish later.
Here it is: https://www.npmjs.com/package/@pinojs/json-colorizer. Could you send a PR to integrate it? Thanks!
Would you like to send a Pull Request to add the support for it? Remember to add unit tests.
Have you tracked down where that error come from? I don't think it comes from pino-pretty.
I don't know to be honest. Maybe something in the http-fastify-complete example?