santinic / how2

AI for the Command Line
https://how2terminal.com
MIT License
5.71k stars 156 forks source link

Changed `devnull` dependency to a specific commit to get fix for EventEmitter issue. #76

Closed goibon closed 6 years ago

goibon commented 7 years ago

This changes the version of devnull in package.json in order to get the change introduced in this PR. This allows how2 to be used with Node v. 7.x.x and addresses #70.

The reason I set it to a specific commit is because they have yet to to bump their version number allowing a regular update through npm. This is not a great solution, but since they haven't updated their version yet even though the PR went through in June we can't count on them doing it any time soon.

A better long term solution may be to switch to a different module instead of devnull.

PaoloCifariello commented 6 years ago

Fixed in #85