{"push_id":2370651250,"size":2,"distinct_size":2,"ref":"refs/heads/phantomjs-user-agent","head":"cbf4ea9174cf5d83350bc66a0c08785aa25e051d","before":"13d84e7a21d0f68b0461bd8f40f55a21396e028c","commits":[{"sha":"08875e743b45be831ddc6740996adbc801a8ed4f","author":{"name":"Aaron Renner","email":"ad7aa93a7c6096a37035ef45342e620f20be1f9c@gmail.com"},"message":"Fix zombie phantomjs processes (#338)\n\nAfter running wallaby as part of a separate application, I confirmed we\r\nwere still leaving around zombie phantom processes. It turns out if we\r\nkill the script's process group, it will kill all of the child processes\r\nas well. This appears to fix the zombie issue once and for all.\r\n\r\nOne thing to note, I kill the process group by using the negative value\r\nof the wrapper script's pid, instead of using `ps axo pid, pgid` to look\r\nup the process group id. This is because I was occasionally running into\r\nissues where the process group lookup wasn't returning a result, causing\r\nmore zombie processes. Hopefully this approach won't cause any issues.\r\n\r\nFixes #224.","distinct":true,"url":"https://api.github.com/repos/joshuataylor/wallaby/commits/08875e743b45be831ddc6740996adbc801a8ed4f"},{"sha":"cbf4ea9174cf5d83350bc66a0c08785aa25e051d","author":{"name":"Josh Taylor","email":"e1f2f2e60c7a529557314b23b73f38e9c3754ed8@gmail.com"},"message":"Merge branch 'master' into phantomjs-user-agent","distinct":true,"url":"https://api.github.com/repos/joshuataylor/wallaby/commits/cbf4ea9174cf5d83350bc66a0c08785aa25e051d"}]}
{"actor":{"display_login":"joshuataylor"}}