-
Notifications
You must be signed in to change notification settings - Fork 423
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
Remove warnings of protobuf #645
Remove warnings of protobuf #645
Conversation
66ff827
to
f62a01a
Compare
Codecov Report
@@ Coverage Diff @@
## main #645 +/- ##
==========================================
+ Coverage 94.65% 94.66% +0.01%
==========================================
Files 203 203
Lines 9322 9322
==========================================
+ Hits 8824 8825 +1
+ Misses 498 497 -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.
Thanks for the fix :)
Thank you! |
@owt5008137 Can you rebase the branch with main, that will allow us to merge the changes. |
4503c9f
to
4ceb351
Compare
@lalitb done. |
Not sure why the CLA verification is stuck for this PR, while it is passing for others. |
Don't know the reason either, but it seems passed after I click the Update button. |
Related to #249
Changes
Remove warnings of protobuf. There is no functional change.
For significant contributions please make sure you have completed the following items:
CHANGELOG.md
updated for non-trivial changes