Skip to content

docs: update after nodejs/node#5756#45

Merged
AndreasMadsen merged 1 commit intonodejs:masterfrom
AndreasMadsen:async-wrap-update-pr-5756
Apr 29, 2016
Merged

docs: update after nodejs/node#5756#45
AndreasMadsen merged 1 commit intonodejs:masterfrom
AndreasMadsen:async-wrap-update-pr-5756

Conversation

@AndreasMadsen
Copy link
Member

nodejs/node#5756 changed setupHooks to take an object as input and added didThrow as an argument to the post hook.


##### didThrow

If the callback threw, but the process was allowed to continue dude to either

This comment was marked as off-topic.

@AndreasMadsen
Copy link
Member Author

@mjbvz thanks, if you give me an LGTM I will just go ahead and merge this.

@mjbvz
Copy link

mjbvz commented Apr 28, 2016

Still learning about this area, but LGTM

@AndreasMadsen AndreasMadsen merged commit 2650b32 into nodejs:master Apr 29, 2016
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.

2 participants