-
Notifications
You must be signed in to change notification settings - Fork 492
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
feat:refactor SDKContext as static. #995
feat:refactor SDKContext as static. #995
Conversation
Codecov Report
@@ Coverage Diff @@
## 2022.0 #995 +/- ##
============================================
+ Coverage 82.89% 82.95% +0.05%
- Complexity 1583 1599 +16
============================================
Files 238 239 +1
Lines 5244 5320 +76
Branches 610 618 +8
============================================
+ Hits 4347 4413 +66
- Misses 540 549 +9
- Partials 357 358 +1
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
028832e
to
0956a57
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
PR Type
Refactoring (no functional changes, no api changes).
Describe what this PR does for and how you did.
Adding the issue link (#xxx) if possible.
fixes #982
fixes #984
Note
Checklist
Checklist (Optional)