Skip to content
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

[Certora] Reverts #390

Merged
merged 36 commits into from
Jan 17, 2024
Merged

[Certora] Reverts #390

merged 36 commits into from
Jan 17, 2024

Conversation

QGarchery
Copy link
Contributor

@QGarchery QGarchery commented Jan 10, 2024

This PRs goal is to verify the condition in which inputs are not valid. In certain case the rule is called *RevertCondition instead of *InputValidation because we are able to specify all the revert conditions simply

@QGarchery QGarchery self-assigned this Jan 10, 2024
@QGarchery QGarchery changed the base branch from main to certora/consistent-pending-values January 10, 2024 13:38
Base automatically changed from certora/consistent-pending-values to main January 10, 2024 15:40
@QGarchery QGarchery marked this pull request as ready for review January 11, 2024 19:09
@QGarchery QGarchery mentioned this pull request Jan 11, 2024
28 tasks
certora/specs/Reverts.spec Show resolved Hide resolved
certora/specs/Reverts.spec Show resolved Hide resolved
certora/specs/Reverts.spec Outdated Show resolved Hide resolved
certora/specs/Reverts.spec Show resolved Hide resolved
certora/specs/Reverts.spec Show resolved Hide resolved
certora/specs/Reverts.spec Show resolved Hide resolved
Copy link
Contributor

@MathisGD MathisGD left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, let's keep my remark in an issue

@MathisGD MathisGD merged commit 74284aa into main Jan 17, 2024
9 checks passed
@MathisGD MathisGD deleted the certora/reverts branch January 17, 2024 11:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants