Skip to content

Add benchmark results to readme #9

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Oct 14, 2020

Conversation

anthonyringoet
Copy link
Contributor

As title states, added benchmark results to readme.

Checklist

@mcollina
Copy link
Member

Can you please change Bourne.parse to secure-json-parse? thanks

@anthonyringoet
Copy link
Contributor Author

Yes, that makes sense. Updated the readme and benchmark files.

@RafaelGSS
Copy link
Member

@mcollina how periodically will update it?

@anthonyringoet the machine used is a virtualized machine or a desktop/notebook personal with possible external impacts?

@anthonyringoet
Copy link
Contributor Author

@RafaelGSS Personal notebook. It is certainly not free of possible external impacts as it's not a standardized testing setup. It's always running some other software.

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 b062cdb into fastify:master Oct 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants