-
Notifications
You must be signed in to change notification settings - Fork 656
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
[Feature] Implement leo execute #2491
Conversation
Codecov Report
@@ Coverage Diff @@
## testnet3 #2491 +/- ##
============================================
+ Coverage 78.90% 79.01% +0.11%
============================================
Files 155 156 +1
Lines 4916 4947 +31
Branches 4915 4947 +32
============================================
+ Hits 3879 3909 +30
- Misses 1037 1038 +1
... and 10 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
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 mostly good to go!
Left a few minor nits that can be addressed at your discretion.
Co-authored-by: d0cd <pranavsaig@gmail.com> Signed-off-by: Collin Chin <16715212+collinc97@users.noreply.github.com>
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.
Works well for me. leo run
is now super-fast, thanks!
leo execute
works too, and prints out a JSON structure.
Is there a switch to turn off the output of the JSON structure?
Upgrades
snarkvm v0.14.5
updatesleo run
significant speedupleo execute
command