Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.75%. Comparing base (1e28be8) to head (7492e28).
Report is 18 commits behind head on main.

@@            Coverage Diff             @@
##             main    #1184      +/-   ##
==========================================
+ Coverage   86.24%   86.75%   +0.50%     
==========================================
  Files          17       17              
  Lines        2436     2423      -13     
  Branches      258      257       -1     
==========================================
+ Hits         2101     2102       +1     
+ Misses        249      238      -11     
+ Partials       86       83       -3     

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

@methane thanks for enabling the CI runs and fixing the spacing!

The complaint of coverage is that it seems the catch-all except BaseException in Connection.connect isn't being hit by any test, right?

I'll try to add a test for that. Also, the code in that except is basically _force_close now, maybe we should simply call that there.

thanks