-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
[WIP] feat: remove SockJS support #5626
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
base: next
Are you sure you want to change the base?
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests.
Additional details and impacted files@@ Coverage Diff @@
## next #5626 +/- ##
==========================================
- Coverage 83.59% 77.06% -6.54%
==========================================
Files 13 11 -2
Lines 2054 1966 -88
Branches 756 735 -21
==========================================
- Hits 1717 1515 -202
- Misses 303 384 +81
- Partials 34 67 +33 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
7fb1abf to
e9d63b4
Compare
Signed-off-by: Sebastian Beltran <[email protected]>
…nd package-lock.json
012357a to
a181f84
Compare
|
@bjohansebas If you will need a help feel free to ping me |
Summary
What kind of change does this PR introduce?
Did you add tests for your changes?
Does this PR introduce a breaking change?
If relevant, what needs to be documented once your changes are merged or what have you already documented?