-
Notifications
You must be signed in to change notification settings - Fork 114
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 kotlin warnings #551
Fix kotlin warnings #551
Conversation
Signed-off-by: Siddhant Deshmukh <[email protected]>
Signed-off-by: Siddhant Deshmukh <[email protected]>
Codecov Report
@@ Coverage Diff @@
## main #551 +/- ##
============================================
- Coverage 75.94% 75.94% -0.01%
- Complexity 2486 2489 +3
============================================
Files 316 316
Lines 14547 14546 -1
Branches 2248 2248
============================================
- Hits 11048 11047 -1
- Misses 2240 2241 +1
+ Partials 1259 1258 -1
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
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.
Thank You for cleaning up the code, does this change require updating any tests or supplementary documentation?
The tests should not need any changes. Where can I check if the documentation needs to be updated? Do I need to manually go over the OpenSearch public documentation? |
* Fix low hanging fruit warnings Signed-off-by: Siddhant Deshmukh <[email protected]> * Revert change causing IT failure Signed-off-by: Siddhant Deshmukh <[email protected]> Signed-off-by: Siddhant Deshmukh <[email protected]>
Signed-off-by: Siddhant Deshmukh [email protected]
Issue #239 , if available:
Description of changes:
CheckList:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.