hapijs/hapi-pino

Upgrade to pino@^7.0.0

Closed this issue · 3 comments

I have just started to use pino directly in addition to hapi-pino, and noticed that hapi-pino depends on an older version of pino@^6.0.0. Now I have two versions of pino in my node_modules, a situation I’d prefer to avoid.

Would it make sense to upgrade hapi-pino to use pino@^7.0.0?

Would you like to send a Pull Request to address this issue? Remember to add unit tests.

Sure, I’ve created a PR, however I am uncertain about adding new unit tests. Isn’t it covered by like 70 tests already?

Thanks!