New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adding a disconnectAsync method #141
base: master
Are you sure you want to change the base?
Conversation
0b3ae01
to
dc1e889
Compare
retest this please |
897ea92
to
3199217
Compare
822034f
to
57a1741
Compare
57a1741
to
d246114
Compare
…or getResultRowForPixel() calls
(cherry picked from commit 00ee08a)
d246114
to
1dbe673
Compare
…unchecked by sending it to console.error. This will make it easier to take the appropriate action on server crashes during test runs.
@vastcharade Can we close this PR for now? |
Yeah, i guess so, particularly since it's so old. Has a |
nope, still no async disconnect from what I can tell. |
@vastcharade Sounds like we should keep it open until those points are resolved. It'd definitely better to add the support you need. Sorry you've had to maintain the separate branch for all this time! |
Merge Checklist
🔧 Issue(s) fixed:
🚬 Smoke Test
🚢 Merge