-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
refactor: client #3129
refactor: client #3129
Conversation
46be16a
to
58ca88f
Compare
|
Also we need improve |
802f69c
to
6f6bfdd
Compare
Codecov Report
@@ Coverage Diff @@
## master #3129 +/- ##
==========================================
+ Coverage 91.95% 94.78% +2.82%
==========================================
Files 37 37
Lines 1292 1207 -85
Branches 349 325 -24
==========================================
- Hits 1188 1144 -44
+ Misses 96 57 -39
+ Partials 8 6 -2
Continue to review full report at Codecov.
|
Weird with coverage 😕 |
I want postpone logger refactor for #2932 |
macos on github actions is hell... |
For Bugs and Features; did you add new tests?
Existing
Motivation / Use-Case
Improve structure, refactor code, allow to run on IE11/IE10/IE9, IE8 is not supported, sorry
Breaking Changes
No
Additional Info
IE11/IE10:
IE9: