-
Notifications
You must be signed in to change notification settings - Fork 29.8k
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
doc: copyedit benchmark/README.md #1970
Conversation
fdebca0
to
8512333
Compare
|
||
1. Run all tests of a given type, for example, buffers | ||
1\. Run all tests of a given type, for example, buffers |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These should probably just be sub-headers.. h3?
8512333
to
c2058a6
Compare
Pushed new commit per feedback from @Fishrock123 changing list to headers |
c2058a6
to
f2043d5
Compare
Pinging the only people listed as collaborators who have commits on this file in the hopes of getting a quick "LGTM" or two: @Fishrock123 @jbergstroem @yosuke-furukawa |
LGTM |
PR-URL: nodejs#1970 Reviewed-By: Johan Bergström <[email protected]>
Merged in 5c2707c |
No description provided.