You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
While playing locally with the event emitter, I noticed the method off didn't exist for me - I found out it was added in node 10 (and of course I'm running an outdated version, as always).
Would it make sense to require Node >= 10 in package.json? Not sure what conflicts this could cause, what version Travis CI is running, etc
The text was updated successfully, but these errors were encountered:
For Travis CI we can probably configure the right or even multiple node.js versions. Also, I think supporting the LTS version (which I think is v8 right now?) is probably better.
Does the off method have a shim or polyfill we can optionally add in?
Then again limiting to just Node >= 10 doesn't seem awful.
While playing locally with the event emitter, I noticed the method
off
didn't exist for me - I found out it was added in node 10 (and of course I'm running an outdated version, as always).Would it make sense to require Node >= 10 in package.json? Not sure what conflicts this could cause, what version Travis CI is running, etc
The text was updated successfully, but these errors were encountered: