Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.68%. Comparing base (b7121e5) to head (99b0c5a).
Report is 861 commits behind head on master.

@@            Coverage Diff             @@
##           master    #8611      +/-   ##
==========================================
- Coverage   97.70%   97.68%   -0.03%     
==========================================
  Files         107      107              
  Lines       33438    33441       +3     
  Branches     3927     3928       +1     
==========================================
- Hits        32672    32666       -6     
- Misses        555      563       +8     
- Partials      211      212       +1     

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

So, I'm now not convinced that wait_for_disconnection() is useful (and possibly not correct). Maybe it should be removed again...?

So, I'm now not convinced that wait_for_disconnection() is useful (and possibly not correct). Maybe it should be removed again...?

I'm not sure its useful, but I've also never had a use case for it. I'd open an issue to provide notice of intent to remove it, and than do so in 30 days if nobody objects.

Backport to 3.10: 💚 backport PR created

✅ Backport PR branch: patchback/backports/3.10/1fcef940e37c93f1dc05b859816affaf7228b2f8/pr-8611

Backported as #8627

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

Backport to 3.11: 💚 backport PR created

✅ Backport PR branch: patchback/backports/3.11/1fcef940e37c93f1dc05b859816affaf7228b2f8/pr-8611

Backported as #8628

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

I'm not sure its useful, but I've also never had a use case for it. I'd open an issue to provide notice of intent to remove it, and than do so in 30 days if nobody objects.

Well, it's only been in a release for a week or so. So, it's incredibly unlikely people are already using it, we can probably just strip in the next 3.10 release before most users have finished upgrading to 3.10.

I'm not sure its useful, but I've also never had a use case for it. I'd open an issue to provide notice of intent to remove it, and than do so in 30 days if nobody objects.

Well, it's only been in a release for a week or so. So, it's incredibly unlikely people are already using it, we can probably just strip in the next 3.10 release before most users have finished upgrading to 3.10.

I agree. No need to wait as its unlikely its being used.