-
Notifications
You must be signed in to change notification settings - Fork 232
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
fix(deps)!: use grpc-js instead of grpc extension #658
Conversation
Codecov Report
@@ Coverage Diff @@
## master #658 +/- ##
==========================================
- Coverage 97.89% 97.79% -0.11%
==========================================
Files 14 14
Lines 856 861 +5
Branches 178 179 +1
==========================================
+ Hits 838 842 +4
Misses 2 2
- Partials 16 17 +1
Continue to review full report at Codecov.
|
@sduskis I'm running some stress tests locally, but as of right now I think we're looking good. I wrote a new test specifically for messages that are 9mb and it appears to be working just fine. |
Fixes #627