Skip to content
This repository has been archived by the owner on Aug 15, 2024. It is now read-only.

Investigate replacing gozala/events module #44

Open
connor4312 opened this issue Oct 12, 2016 · 3 comments
Open

Investigate replacing gozala/events module #44

connor4312 opened this issue Oct 12, 2016 · 3 comments

Comments

@connor4312
Copy link

connor4312 commented Oct 12, 2016

Currently events is polyfilled with gozala/events. However, it is far behind the latest Node version, and perhaps more concerningly its unit tests are failing and they are not run in a wide array of browsers, so upgrading it is dangerous (though I have a PR in to do so).

Unfortunately it doesn't look like there's other solid isomorphic event implementations which match the Node API, at least none that my Google-fu could find. I would be happy to spend a weekend getting something drafted up if there's interest in updating this; an updated implementation to be tested both in Node via and in browsers via Browserstack or SauceLabs.

@jhnns
Copy link
Member

jhnns commented Nov 3, 2016

Do you have any specific problem with gozala/events? Where is the API not in parity with node.js?

@connor4312
Copy link
Author

Missing 'features' include:

It also does not include m/any bug fixes or optimizations applied to the events module since the last big import in 2013.

@jhnns
Copy link
Member

jhnns commented Nov 4, 2016

Ok, maybe let's give the maintainer some time

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants