Skip to content

Update testSuccessfulRetry and code to make it pass.#150

Closed
danielt1263 wants to merge 1 commit intoCombineCommunity:mainfrom
danielt1263:bug-fix/retryWhen
Closed

Update testSuccessfulRetry and code to make it pass.#150
danielt1263 wants to merge 1 commit intoCombineCommunity:mainfrom
danielt1263:bug-fix/retryWhen

Conversation

@danielt1263
Copy link
Contributor

Update based on observation by @Kn1kt in issue #148

@codecov
Copy link

codecov bot commented Nov 13, 2022

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.49%. Comparing base (d7b896f) to head (b8ec4a3).
⚠️ Report is 18 commits behind head on main.

Additional details and impacted files
@@            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     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@robnadin
Copy link

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

@WoodyHang
Copy link

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

@danielt1263
Copy link
Contributor Author

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?

@freak4pc
Copy link
Member

I'm going to close this since it's super-old and I'm trying to clean up this repo. If this is still an issue / or something you'd like to pursue, feel free to comment and we can re-discuss. Thanks!

@freak4pc freak4pc closed this Jan 20, 2026
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.

4 participants