Skip to content
This repository has been archived by the owner on Dec 15, 2022. It is now read-only.

Update README removing replacing pino with pino-pretty. pino usage as cli was deprecated #25

Closed
pragmaticivan opened this issue Dec 28, 2018 · 1 comment
Closed

Update README removing replacing pino with pino-pretty. pino usage as cli was deprecated #25

pragmaticivan opened this issue Dec 28, 2018 · 1 comment

Comments

@pragmaticivan
Copy link
Contributor

No description provided.

@mcollina
Copy link
Member

Thanks! Would you like to send a PR?

pragmaticivan added a commit to pragmaticivan/express-pino-logger that referenced this issue Dec 28, 2018
@pragmaticivan
Fixes pinojs#25
e34827d 
## Why is this change necessary?
pino is deprecated as cli
## How does it address the issue?
it uses `pino-pretty` in the example instead.
## What side effects does this change have?
No side effect.
@pragmaticivan pragmaticivan mentioned this issue Dec 28, 2018
Merged
mcollina added a commit that referenced this issue Dec 29, 2018
@mcollina
Merge pull request #26 from pragmaticivan/feature/25-update-readme-fo…
a75f796 
…r-pino-pretty

[#25] Updates example replacing pino with pino-pretty
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mcollina @pragmaticivan