-
Notifications
You must be signed in to change notification settings - Fork 16
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 a script for circuit compilation and verifier generation #27
add a script for circuit compilation and verifier generation #27
Conversation
Signed-off-by: Chengxuan Xing <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #27 +/- ##
==========================================
+ Coverage 44.52% 45.09% +0.56%
==========================================
Files 14 14
Lines 530 530
==========================================
+ Hits 236 239 +3
+ Misses 255 253 -2
+ Partials 39 38 -1 ☔ View full report in Codecov by Sentry. |
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 great @Chengxuan! something that I've been wanting to do as well. will be super useful!
Signed-off-by: Chengxuan Xing <[email protected]>
Signed-off-by: Chengxuan Xing <[email protected]>
auto rename contract
Signed-off-by: Chengxuan Xing <[email protected]>
Signed-off-by: Chengxuan Xing <[email protected]>
Signed-off-by: Chengxuan Xing <[email protected]>
@jimthematrix thanks for the review comments. Switched the script to an NPM script with all comment addressed |
Signed-off-by: Chengxuan Xing <[email protected]>
Signed-off-by: Chengxuan Xing <[email protected]>
Add a bash script to automate what's documented for circuit compilation, verification key generation and verifier library generation.
The
npm run gen
script by default processes up to 10 (configurable) circuits in parallel which hugely reduce the dev cycle.