-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
[CLI][Prover][trivial] set option compile_verify_code
to true when running prover using V2 flag
#14900
Conversation
⏱️ 2h 16m total CI duration on this PR
🚨 1 job on the last run was significantly faster/slower than expected
|
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @rahxephon89 and the rest of your teammates on Graphite |
compile_verify_code
to true to prover
compile_verify_code
to true to prover compile_verify_code
to true to prover
compile_verify_code
to true to prover compile_verify_code
to true when running prover using V2 flag
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #14900 +/- ##
=========================================
- Coverage 60.0% 60.0% -0.1%
=========================================
Files 856 856
Lines 210624 210625 +1
=========================================
Hits 126573 126573
- Misses 84051 84052 +1 ☔ View full report in Codecov by Sentry. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
✅ Forge suite
|
✅ Forge suite
|
✅ Forge suite
|
Description
This PR set
options.compile_verify_code
to true when building the model for prover with V2 flag so thataptos move prove --move-2
will verify functions with the attribute#[verify_only]
.Close #14895.
How Has This Been Tested?
Manual check by running
aptos move prove --move-2
.Key Areas to Review
Type of Change
Which Components or Systems Does This Change Impact?
Checklist