search

tim-kos / node-retry

Abstraction for exponential and custom retry strategies for failed operations.
Other
1.22k stars 80 forks source link

Handle errors despite timeouts #71

Closed cdeevfrr closed 3 years ago

cdeevfrr commented 5 years ago

Closes #70

codecov[bot] commented 5 years ago

Codecov Report

Merging #71 into master will increase coverage by 0.06%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #71      +/-   ##
==========================================
+ Coverage    90.2%   90.27%   +0.06%     
==========================================
  Files           2        2              
  Lines         143      144       +1     
==========================================
+ Hits          129      130       +1     
  Misses         14       14
Impacted Files Coverage Δ
lib/retry_operation.js 86.17% <100%> (+0.14%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 88548b9...e3865c7. Read the comment docs.

cdeevfrr commented 4 years ago

Has anyone had a chance to look at this?

tim-kos commented 4 years ago

I will look into it soon. Very busy with Transloadit things at the moment, sorry.

On Tue, Mar 3, 2020 at 2:47 PM Danny Rivers notifications@github.com wrote:

Has anyone had a chance to look at this?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/tim-kos/node-retry/pull/71?email_source=notifications&email_token=AAADVHNHHWZNYMQDAPWY3QDRFUDAHA5CNFSM4JJGXYIKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENTRMYY#issuecomment-593958499, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAADVHO2AYPSM2Q6FPJK34LRFUDAHANCNFSM4JJGXYIA .

tim-kos commented 3 years ago

Great, thank you!