Codecov Report

Merging #448 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #448   +/-   ##
=======================================
  Coverage   56.94%   56.94%           
=======================================
  Files           6        6           
  Lines         734      734           
  Branches       42       42           
=======================================
  Hits          418      418           
  Misses        307      307           
  Partials        9        9

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cedbadc...648ad24. Read the comment docs.

Thanks for the PR @c-goosen! I'd like to test a few things before merging this PR, but I'm hoping to get to this very soon.

@RodneyU215 no problem. Let me know if you need any help or additional tests. I've run into this a couple of times. I am going to do an additional PR for documentation around async and this library.

@RodneyU215 also if it helps, this is running in my infra already, going to prod in the next couple of days, I am pretty sure from my side that it works.

@c-goosen Thanks for this PR and for helping to triage #429. It's led me to uncover a deeper issue with how I initially implemented the interactions with the event loop. I do not believe this particular change will be necessary once #466 is merged. Please take a look if you have time.