-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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: check obj type in protobufBinding #2851
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2851 +/- ##
=======================================
Coverage 98.75% 98.75%
=======================================
Files 41 41
Lines 3054 3058 +4
=======================================
+ Hits 3016 3020 +4
Misses 26 26
Partials 12 12
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
build fail |
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
* fix: check obj type in protobufBinding * fix: UnitTest for invalid proto obj
Change-Id: I194f7010835974b489b497c6d1f1b4e47b99864e
master
Solve #2850