{"push_id":1375759110,"size":8,"distinct_size":8,"ref":"refs/heads/master","head":"b9f6a2dc059a1062776133f3d4fd848c4da7d150","before":"758ca8d179dd2ddf22c680d812870a12fc9430cd","commits":[{"sha":"2bd850b370cee688aaf377bde41f1b0a0cb19edd","author":{"name":"James M Snell","email":"b0bb7a8fecaaa9b5986f6f19a6b09d78632bd248@gmail.com"},"message":"src: fix use of uninitialized variable\n\nVariable was uninitialized in 72547fe28d\nInitialize the variable and add a static_check\n\nPR-URL: https://github.com/nodejs/node/pull/9281\nReviewed-By: Ben Noordhuis <info@bnoordhuis.nl>","distinct":true,"url":"https://api.github.com/repos/jasnell/node/commits/2bd850b370cee688aaf377bde41f1b0a0cb19edd"},{"sha":"fd54df1e674f408c6c1bc314726e23ef58da15f6","author":{"name":"Rich Trott","email":"c1321c10e42359e7caa6e441829d03e9d30a4048@gmail.com"},"message":"test: fix flaky test-fs-watch-recursive on OS X\n\nThe test was sometimes timing out due to a race condition. In OS X,\nevents for `fs.watch()` might only start showing up after a delay. This\nis a limitation of the operating system. To work around that, there was\na timer in the test that delayed the writing of the file by 100ms.\nHowever, sometimes that was not enough, and so the event never fired,\nand the test timed out.\n\nChange the timer to an interval so that it fires repeatedly until it is\npicked up. This change only affects OS X.\n\nFixes: https://github.com/nodejs/node/issues/8511\nPR-URL: https://github.com/nodejs/node/pull/9303\nReviewed-By: Colin Ihrig <cjihrig@gmail.com>\nReviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>","distinct":true,"url":"https://api.github.com/repos/jasnell/node/commits/fd54df1e674f408c6c1bc314726e23ef58da15f6"},{"sha":"2c6ca32e8bcc6759a704bb525df71ac7b3b151ca","author":{"name":"Rich Trott","email":"c1321c10e42359e7caa6e441829d03e9d30a4048@gmail.com"},"message":"test: run all of test-timers-blocking-callback\n\nThe test has two test cases, but only the first was being run due to a\nsmall bug. This change fixes the bug.\n\nPR-URL: https://github.com/nodejs/node/pull/9305\nReviewed-By: Julien Gilli <jgilli@nodejs.org>\nReviewed-By: Colin Ihrig <cjihrig@gmail.com>","distinct":true,"url":"https://api.github.com/repos/jasnell/node/commits/2c6ca32e8bcc6759a704bb525df71ac7b3b151ca"},{"sha":"07cc9dfd05fa49e0b33b1b4dd74a36138f954e96","author":{"name":"Rod Vagg","email":"b188db099f2f059a691751fd624f34546de70910@vagg.org"},"message":"benchmark: add microbenchmarks for ES Map\n\nPR-URL: https://github.com/nodejs/node/pull/7581\nReviewed-By: James M Snell <jasnell@gmail.com>\nReviewed-By: Franziska Hinkelmann <ranziska.hinkelmann@gmail.com>","distinct":true,"url":"https://api.github.com/repos/jasnell/node/commits/07cc9dfd05fa49e0b33b1b4dd74a36138f954e96"},{"sha":"d0ed173d6d3e1ee394901da167cb836ce1c83772","author":{"name":"cjihrig","email":"b0a62289a2635542c30e3402b36d8cbeb000fe3e@gmail.com"},"message":"test: add child_process customFds test\n\nThis commit adds a test for spawn()'s deprecated customFds option.\n\nPR-URL: https://github.com/nodejs/node/pull/9307\nReviewed-By: Rich Trott <rtrott@gmail.com>\nReviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>\nReviewed-By: James M Snell <jasnell@gmail.com>","distinct":true,"url":"https://api.github.com/repos/jasnell/node/commits/d0ed173d6d3e1ee394901da167cb836ce1c83772"},{"sha":"ec7c27f4cb80c3f2600ec7e024276c2831273a47","author":{"name":"cjihrig","email":"b0a62289a2635542c30e3402b36d8cbeb000fe3e@gmail.com"},"message":"child_process: remove unreachable code\n\n_convertCustomFds() is only called from normalizeSpawnArguments(),\nwhich always provides an options object. Therefore, there is no\nneed to check for options in _convertCustomFds().\n\nPR-URL: https://github.com/nodejs/node/pull/9307\nReviewed-By: Rich Trott <rtrott@gmail.com>\nReviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>\nReviewed-By: James M Snell <jasnell@gmail.com>","distinct":true,"url":"https://api.github.com/repos/jasnell/node/commits/ec7c27f4cb80c3f2600ec7e024276c2831273a47"},{"sha":"6ef636c0c93e249f9da8bf060dc537667b9d3595","author":{"name":"Rich Trott","email":"c1321c10e42359e7caa6e441829d03e9d30a4048@gmail.com"},"message":"test: fix freebsd10-64 CI failures\n\nRemove unneeded timers from some tests and move others from parallel\ntesting to sequential testing.\n\nThis is to resolve test failures on freebsd10-64 on CI. The failures\nare all due to timers firing later than expected. Timers firing later\nthan they are set for can happen on resource-constrained hosts and is\nnot a bug.\n\nIn general, it may be wise to put tests that depend on timing into\nsequential testing rather than parallel testing, as the timing can\nbe affected by other simultaneously-running test processes.\n\nFixes: https://github.com/nodejs/node/issues/8041\nFixes: https://github.com/nodejs/node/issues/9227\nPR-URL: https://github.com/nodejs/node/pull/9317\nReviewed-By: Colin Ihrig <cjihrig@gmail.com>\nReviewed-By: Anna Henningsen <anna@addaleax.net>\nReviewed-By: Gibson Fahnestock <gibfahn@gmail.com>\nReviewed-By: Julien Gilli <jgilli@nodejs.org>\nReviewed-By: Johan Bergstrom <bugs@bergstroem.nu>\nReviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>","distinct":true,"url":"https://api.github.com/repos/jasnell/node/commits/6ef636c0c93e249f9da8bf060dc537667b9d3595"},{"sha":"b9f6a2dc059a1062776133f3d4fd848c4da7d150","author":{"name":"Isobel Redelmeier","email":"009edd2213cc07ed164319015e278e9d71450bd5@gmail.com"},"message":"test: clean up dgram-broadcast-multi-process test\n\nUse assert.strictEqual() instead of assert.equal(),\n=== instead of ==, and const instead of var.\n\nPR-URL: https://github.com/nodejs/node/pull/9308\nReviewed-By: Rich Trott <rtrott@gmail.com>\nReviewed-By: Bryan English <bryan@bryanenglish.com>\nReviewed-By: Colin Ihrig <cjihrig@gmail.com>\nReviewed-By: James M Snell <jasnell@gmail.com>","distinct":true,"url":"https://api.github.com/repos/jasnell/node/commits/b9f6a2dc059a1062776133f3d4fd848c4da7d150"}]}
{"actor":{"display_login":"jasnell"}}