node-flowdock icon indicating copy to clipboard operation
node-flowdock copied to clipboard

Node-Flowdock doesn't work in Node 7

Open pimterry opened this issue 8 years ago • 0 comments

In Node 7, process.EventEmitter doesn't exist (and in Node 6 it's deprecated, and will print a warning), but Session here extends it. See https://github.com/nodejs/node/pull/6862 for more details.

It'd be nice if the engines field in node-flowdock's package.json caught this, but it'd be even better if this worked. You could plausibly move to const EventEmitter = process.EventEmitter || require('events').EventEmitter if you want to keep backwards compatibility too.

pimterry avatar Feb 10 '17 13:02 pimterry