New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix CustomEventEmitter #1245

Merged
merged 1 commit into from Jan 13, 2014

Conversation

2 participants
@saschagehlich
Contributor

saschagehlich commented Jan 13, 2014

Make sure that objects passed to CustomEventEmitter's error event
are boiled down to real Error instances so that node's EventEmitter
does not give us this nasty "Uncaught, unspecified error" thing.

Fix CustomEventEmitter
Make sure that objects passed to CustomEventEmitter's `error` event
are boiled down to real Error instances so that node's EventEmitter
does not give us this nasty "Uncaught, unspecified error" thing.

mickhansen added a commit that referenced this pull request Jan 13, 2014

@mickhansen mickhansen merged commit 98b4637 into sequelize:master Jan 13, 2014

1 check passed

default The Travis CI build passed
Details

@saschagehlich saschagehlich deleted the saschagehlich:bugfix/customeventemitter-fix branch Jan 26, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment