README is confusing on the constructor usage

[rprieto]

The README at https://www.npmjs.com/package/pino#constructor says pino([stream], [opts]). I thought this meant we can call pino(process.stdout, {extreme: true}). However that doesn't work, which makes sense when we look at the code:

function pino (opts, stream) {
  if (opts && opts._writableState) {
    stream = opts
    opts = null
  }
}

Should the README say pino([opts], [stream]) (reversed)?

[mcollina]

Thanks, fixed!

[github-actions]

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