Skip to content
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

Apiary reporter not throwing anymore, only passing error to callback #275

Merged
merged 2 commits into from
Sep 21, 2015

Conversation

miiila
Copy link
Contributor

@miiila miiila commented Sep 10, 2015

Introducing _performRequestAsync to return callback filled with an error instead of fatal error in Apiary reporter.

Introduced change is triggered by need of assertion failure just after reporter start.

@w-vi
Copy link
Member

w-vi commented Sep 21, 2015

@netmilk ... possibly squash to single commit? Or two commits one from miiila and one from you?

@netmilk netmilk force-pushed the miiila/return-authentication-error-via-js-api branch from 591781c to a393509 Compare September 21, 2015 14:20
@netmilk netmilk changed the title Replace fatal error with error filled callback in apiary reporter Apiary reporter not throwing anymore, only passing error to callback Sep 21, 2015
@netmilk
Copy link
Contributor

netmilk commented Sep 21, 2015

@WVI, squashed.

Seal of approval?

@w-vi
Copy link
Member

w-vi commented Sep 21, 2015

@netmilk approved!

w-vi added a commit that referenced this pull request Sep 21, 2015
…ror-via-js-api

Apiary reporter not throwing anymore, only passing error to callback
@w-vi w-vi merged commit 451146c into master Sep 21, 2015
@netmilk netmilk mentioned this pull request Dec 19, 2015
@pksunkara pksunkara deleted the miiila/return-authentication-error-via-js-api branch June 17, 2016 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants