Skip to content
This repository has been archived by the owner on Oct 17, 2020. It is now read-only.

Consider another EventEmitter #10

Open
binarykitchen opened this issue Feb 4, 2014 · 2 comments
Open

Consider another EventEmitter #10

binarykitchen opened this issue Feb 4, 2014 · 2 comments

Comments

@binarykitchen
Copy link

Hi Thomas

No offense but I recommend to drop your event emitter https://github.com/thomaswelton/bower-event-emitter in favor of this one https://github.com/Wolfy87/EventEmitter

This because yours comes without tests and is not often updated. The other EventEmitter is great and widely used.

Currently I have too many different Event Emitter implementations in my bower directory, bloating up the file size.

See metafizzy/outlayer#11 to get an idea.

Do you think you could switch the Event Emitter? That would be great!

@thomaswelton
Copy link
Owner

I wouldn't be adverse to this. I use my EventEmitter in a few projects, but I was copied from a Gist and I can't get hold of the author to get License info. Which was an issue when I needed to use this on a commercial project.
It looks like the repo you suggested is the most popular EventEmmiter and MIT license. So I'd do the conversion next time I need to use this.

@binarykitchen
Copy link
Author

@thomaswelton Yes please do and switch to that popular EventEmitter whenever you have time, thanks!

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