[Feature] Handle async onRetry

vercel / async-retry

Retrying made simple, easy and async

https://npmjs.com/async-retry

MIT License

1.85k stars 53 forks source link

Closed whitecrownclown closed 3 months ago

whitecrownclown commented 5 years ago

Hello,

This PR is related to #43

I'm not sure about this current implementation, i'd be glad to get someone to review this. (if you don't mind @leo ) A couple of things:

Name of the flag, right now it's awaitOnRetry;
Should it have different opts object or be documented separately from node-retry opts ?
~~Currently using async await, not sure of the targeted browsers, but i assume promises would be more acceptable, will follow up on this if it's a concern;~~

Thanks!

vercel[bot] commented 5 years ago

This pull request is automatically deployed with Now. To access deployments, click Details below or on the icon next to each push.

christiansaiki commented 4 years ago

Any chance of getting this merged? Thanks!