Closed DanDeMicco closed 10 years ago
woohoo it passed. Finally got it!
Yeah sure that sounds like a better, more future proof approach (as long as the log structure doesn't change), should I do the above change and submit another pull request? @JonAbrams @mikaelhm
No need for a new PR, just add a commit to this one.
@JonAbrams should be good to go now! see the latest commit
Nice, thanks!
@JonAbrams Giving this one last (4th) shot for tonight. Apparently node doesn't like the chaining of .on's. Ran npm test on this one and all good. Hopefully. #11