-
Notifications
You must be signed in to change notification settings - Fork 493
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
Add batch verification #2578
Add batch verification #2578
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2578 +/- ##
=======================================
Coverage 47.07% 47.07%
=======================================
Files 349 350 +1
Lines 55858 55925 +67
=======================================
+ Hits 26294 26327 +33
- Misses 26617 26646 +29
- Partials 2947 2952 +5
Continue to review full report at Codecov.
|
…and into add-batch-verification
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.
looks good. two minor comments, but neither of them would be a blocker.
…and into add-batch-verification
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.
Two small comments, but looks good otherwise!
Add the framework for supporting batch verification.
Summary
Test Plan