CombineExt icon indicating copy to clipboard operation
CombineExt copied to clipboard

Update `testSuccessfulRetry` and code to make it pass.

Open danielt1263 opened this issue 2 years ago • 4 comments

Update based on observation by @Kn1kt in issue #148

danielt1263 avatar Nov 13 '22 20:11 danielt1263

Codecov Report

Merging #150 (b8ec4a3) into main (d7b896f) will increase coverage by 0.02%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #150      +/-   ##
==========================================
+ Coverage   95.47%   95.49%   +0.02%     
==========================================
  Files          72       72              
  Lines        4284     4287       +3     
==========================================
+ Hits         4090     4094       +4     
+ Misses        194      193       -1     
Impacted Files Coverage Δ
Sources/Operators/RetryWhen.swift 89.74% <100.00%> (-0.26%) :arrow_down:
Tests/RetryWhenTests.swift 94.66% <100.00%> (+1.70%) :arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

codecov[bot] avatar Nov 13 '22 20:11 codecov[bot]

What's the status of this and #151? Any reason they are not merged?

robnadin avatar Aug 24 '23 09:08 robnadin

Again, why not merge, I need to use this operator now @danielt1263

WoodyHang avatar Oct 10 '23 02:10 WoodyHang

Again, why not merge, I need to use this operator now @danielt1263

I don't think I have control over when this gets merged. Maybe @freak4pc could chime in here?

danielt1263 avatar Oct 10 '23 11:10 danielt1263