-
Notifications
You must be signed in to change notification settings - Fork 6
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
Added certora CI workflow [PE-11] #304
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #304 +/- ##
=======================================
Coverage 95.03% 95.03%
=======================================
Files 75 75
Lines 1773 1773
Branches 235 235
=======================================
Hits 1685 1685
Misses 29 29
Partials 59 59 ☔ View full report in Codecov by Sentry. 🚨 Try these New Features:
|
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.
Not sure we're actually changing any of it?
Did they write a proof for the mitigated front-running attack? Didn't see it, but the PR is massive.
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
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.
👍
Description
This PR integrates the certora verification into the CI. The prover would run on any new changes being pushed to the repo.
Here is the original PR created by the Certora team. We have just recreated the original PR.
Notice
Pull Request Type
Breaking changes (if applicable)
Testing