benchmark: add --format csv option by adrian-nitu-92 · Pull Request #7961

benchmark: add --format csv option by adrian-nitu-92 · Pull Request #7961 · nodejs/node

added the benchmark

Issues and PRs related to the benchmark subsystem.

label

Aug 3, 2016

Trott added a commit to Trott/io.js that referenced this pull request

Aug 6, 2016

Trott mentioned this pull request

Aug 6, 2016

2 tasks

Trott added a commit to Trott/io.js that referenced this pull request

Aug 9, 2016

Refs: nodejs#7961 (comment)
PR-URL: nodejs#8000
Reviewed-By: Brian White <mscdex@mscdex.net>
Reviewed-By: Andreas Madsen <amwebdk@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>

cjihrig pushed a commit that referenced this pull request

Aug 10, 2016

Refs: #7961 (comment)
PR-URL: #8000
Reviewed-By: Brian White <mscdex@mscdex.net>
Reviewed-By: Andreas Madsen <amwebdk@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>

I noticed some typos and the lack of {} following an if.

Signed-off-by: Adrian Nitu <adrian.nitu@intel.com>

Node documentation recommends using process.exitCode = x and returning
as a way to exit.

Signed-off-by: Adrian Nitu <adrian.nitu@intel.com>

Added the option of using --format csv when outputting data.

Signed-off-by: Adrian Nitu <adrian.nitu@intel.com>