delta/node-new.git/benchmark/child_process, branch Remove-Python-3-tests

benchmark: refactor for consistent style

2019-02-07T06:18:31+00:00

Code in benchmark directory sometimes uses `function () {}` for
anonymous callbacks and sometimes uses `() => {}`. Multi-line arrays
sometimes have a trailing comma and sometimes do not. Update to always
use arrow functions for anonymous callbacks and trailing commas for
multiline arrays.

PR-URL: https://github.com/nodejs/node/pull/25944
Reviewed-By: Anto Aravinth 
Reviewed-By: James M Snell

tools: capitalize sentences

2018-12-10T16:07:18+00:00

This adds the `capitalized-comments` eslint rule to verify that
actual sentences use capital letters as starting letters. It ignores
special words and all lines below 62 characters.

PR-URL: https://github.com/nodejs/node/pull/24808
Reviewed-By: Sam Ruby 
Reviewed-By: Trivikram Kamat 
Reviewed-By: Ujjwal Sharma 
Reviewed-By: Sakthipriyan Vairamani

benchmark: remove unused catch bindings

2018-11-06T15:59:23+00:00

PR-URL: https://github.com/nodejs/node/pull/24079
Reviewed-By: Sam Roberts 
Reviewed-By: Wyatt Preul

benchmark: (child_process) use destructuring

2018-01-23T00:29:32+00:00

PR-URL: https://github.com/nodejs/node/pull/18250
Reviewed-By: Matteo Collina 
Reviewed-By: James M Snell

benchmark: var to const

2017-09-20T00:14:38+00:00

PR-URL: https://github.com/nodejs/node/pull/13757
Reviewed-By: Matteo Collina 
Reviewed-By: James M Snell 
Reviewed-By: Luigi Pinca 
Reviewed-By: Gibson Fahnestock 
Reviewed-By: Tobias Nießen

tools,benchmark: use stricter indentation linting

2017-06-27T13:17:06+00:00

Enable stricter indentation rules for benchmark code.

PR-URL: https://github.com/nodejs/node/pull/13895
Reviewed-By: Colin Ihrig 
Reviewed-By: Roman Reiss 
Reviewed-By: James M Snell 
Reviewed-By: Refael Ackermann 
Reviewed-By: Michaël Zasso

test,benchmark: stabilize child-process

2017-06-07T20:27:00+00:00

also some cleanup

PR-URL: https://github.com/nodejs/node/pull/13457
Refs: https://github.com/nodejs/node/issues/12817
Reviewed-By: Tobias Nießen

test: fix flakyness with `yes.exe`

2017-05-06T12:20:07+00:00

PR-URL: https://github.com/nodejs/node/pull/12821
Fixes: https://github.com/nodejs/node/issues/12817
Refs: https://github.com/nodejs/node/pull/12658
Reviewed-By: Rich Trott 
Reviewed-By: Colin Ihrig 
Reviewed-By: Joyee Cheung 
Reviewed-By: James M Snell

benchmark: terminate child process on Windows

2017-04-28T23:18:35+00:00

test-benchmark-child-process failures reveal that
child-process-exec-stdout benchmark sometimes leaves around a stray
yes.exe process. Add code to terminate the process.

PR-URL: https://github.com/nodejs/node/pull/12658
Ref: https://github.com/nodejs/node/issues/12560
Reviewed-By: Refael Ackermann 
Reviewed-By: Bartosz Sosnowski 
Reviewed-By: James M Snell

test: add test for child_process benchmark

2017-04-19T14:58:26+00:00

PR-URL: https://github.com/nodejs/node/pull/12326
Ref: https://github.com/nodejs/node/issues/12068
Reviewed-By: Vse Mozhet Byt 
Reviewed-By: James M Snell 
Reviewed-By: Colin Ihrig 
Reviewed-By: Alexey Orlenko