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

[#25] Updates example replacing pino with pino-pretty #26

Merged

Conversation

pragmaticivan
Copy link
Contributor

@pragmaticivan pragmaticivan commented Dec 28, 2018

Fixes #25

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.

## 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 changed the title Fixes #25 #25 Updates example replacing pino with pino-pretty Dec 28, 2018
@pragmaticivan pragmaticivan changed the title #25 Updates example replacing pino with pino-pretty [#25] Updates example replacing pino with pino-pretty Dec 28, 2018
Copy link
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mcollina mcollina merged commit a75f796 into pinojs:master Dec 29, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants