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

Better readme #4

Merged
merged 2 commits into from
Apr 8, 2016
Merged

Better readme #4

merged 2 commits into from
Apr 8, 2016

Conversation

mcollina
Copy link
Member

@mcollina mcollina commented Apr 7, 2016

@davidmarkclements please review :)
I think this one is done as well!

which is a usecase-specific logger. With
[extreme](https://github.com/mcollina/pino#extreme) mode enable, it is
faster than morgan.
From our [benchmarks](#benchmarks), it is the fastest logger in town for
Copy link
Member

Choose a reason for hiding this comment

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

<nitpick> "it's" instead of "it is" would read a little better here </nitpick>

@davidmarkclements
Copy link
Member

did a rebase because of conflicts - so e060299 is actually my edits

merge if you're happy :)

@mcollina mcollina merged commit a6c9cbe into master Apr 8, 2016
@mcollina mcollina deleted the better-readme branch April 8, 2016 09:50
@mcollina
Copy link
Member Author

mcollina commented Apr 8, 2016

Superb!

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