fix: Moves pino-pretty from devDependencies to dependencies

[spa5k]

Fixes - #1102

More explanation on the issue.

[micalevisk]

pino-pretty is not mandatory when using pino. So to me this change doesn't make sense since some users will end up installing things that they won't use.

[kibertoad]

I think better change would be a more helpful error message when trying to use pino-pretty when one is not installed, instructing user to add it as a dependency.

[jsumners]

Thank you for taking the time to submit a contribution, but we do not include pino-pretty as a hard dependency for a reason. We do not promote prettification in production and our docs are very clear that you must install the module on your own if you want in-process prettification.

[spa5k]

Can we add it to peerDependenciesMeta then?

"peerDependenciesMeta": {
  "pino-pretty": {
    "optional": true
  }
}

@jsumners

[jsumners]

Can we add it to peerDependenciesMeta then?

"peerDependenciesMeta": {
  "pino-pretty": {
    "optional": true
  }
}

@jsumners

Please follow #1106.

[github-actions]

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.