-
Notifications
You must be signed in to change notification settings - Fork 162
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
Dispose of WebSocket when done with it #671
Conversation
Can you push this to a preview build so I can test on production? |
@sungam3r Can you review this quickly? |
@sungam3r Merging now so @STRATZ-Ken can test it. Please comment when you get a chance. The API does change, but it's a non-breaking change. |
Published as nuget version |
Reviewed. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #671 +/- ##
==========================================
- Coverage 54.79% 54.77% -0.03%
==========================================
Files 64 64
Lines 1584 1592 +8
Branches 158 158
==========================================
+ Hits 868 872 +4
- Misses 662 666 +4
Partials 54 54 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
No description provided.