search

julien-c / epub

node.js epub reader
https://www.npmjs.com/package/epub
Other
335 stars 307 forks source link

Comply with es6 and remove dependency of utils #37

Closed Smarticles101 closed 6 years ago

julien-c commented 6 years ago

Hi, thanks for your contribution. I would merge something like this if you manage to get rid of the es-event-emitter dependency.

Thanks!

Smarticles101 commented 6 years ago

@julien-c oops! I forgot I had made this pull request when I pushed that (it doesn’t work anyways). I’ll go ahead and remove that commit when I get a chance.

Smarticles101 commented 6 years ago

@julien-c I went ahead and removed that commit, so the es-event-emitter dependency should be gone!

julien-c commented 6 years ago

Thanks! Merging this right now, and I'll ship a new version.

Next step would be to move to Promises and async/await :)

qqaimh commented 5 years ago

Thanks! Merging this right now, and I'll ship a new version.

Next step would be to move to Promises and async/await :)

Hello, when will support Promises and async/await ?