[ThosRTanner]

Fixes #828 (oddly)

Depends on #801

Why is this needed?

This makes the type checking safer and allows us to remove a lot of casting.

Proposed Changes

• Make all the SequentialHandlers inherit from the IHandler Protocol
• Clean up the IAsyncResult protocol and uses of it.
• ...

Does this PR introduce any breaking change?

No.

[codecov]

Codecov Report

❌ Patch coverage is 72.88136% with 48 lines in your changes missing coverage. Please review.
✅ Project coverage is 90.33%. Comparing base (f54c4fd) to head (03b8760).
⚠️ Report is 3 commits behind head on master.

Files with missing lines	Patch %	Lines
kazoo/handlers/gevent.py	8.82%	31 Missing ⚠️
kazoo/handlers/eventlet.py	0.00%	16 Missing ⚠️
kazoo/protocol/connection.py	93.33%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #829      +/-   ##
==========================================
- Coverage   94.90%   90.33%   -4.57%     
==========================================
  Files          27       27              
  Lines        3811     3799      -12     
==========================================
- Hits         3617     3432     -185     
- Misses        194      367     +173     

☔ View full report in Codecov by Harness.
📢 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.

[ceache]

I like cleanup too, thanks for this.
Could you make sure the commit message follow the guideline.

I would also prefer if you put the content in the commit message itself instead of referring to the PR.

E.g. "chore(typing): clean up some of the workarounds from the initial PR
See PR for details
"

Git commit log is our true history, not so much github PRs.

[ceache]

There is also opportunity for squashing a couple of these commits, I think.

[ThosRTanner]

I like cleanup too, thanks for this. Could you make sure the commit message follow the guideline.

I would also prefer if you put the content in the commit message itself instead of referring to the PR.

E.g. "chore(typing): clean up some of the workarounds from the initial PR See PR for details "

Git commit log is our true history, not so much github PRs.

Yeah, sorry, nowhere near finished on this. hence why it's draft