Skip to content

Remove Hardcoded Min Delay#62

Closed
ElliotFriedman wants to merge 8 commits intomainfrom
feat/remove-hardcoded-min-delay
Closed

Remove Hardcoded Min Delay#62
ElliotFriedman wants to merge 8 commits intomainfrom
feat/remove-hardcoded-min-delay

Conversation

@ElliotFriedman
Copy link
Contributor

This removes the enforced lower bound on minimum delay.

This is intentional and we accept all risks that come with this.

https://prover.certora.com/output/651303/65ef282431474a29b9bf49cd58028138?anonymousKey=c9db8981c953b7ab947389a7d0c94d14881146cf

Signed-off-by: Elliot <elliotfriedman3@gmail.com>
Signed-off-by: Elliot <elliotfriedman3@gmail.com>
Signed-off-by: Elliot <elliotfriedman3@gmail.com>
Signed-off-by: Elliot <elliotfriedman3@gmail.com>
Signed-off-by: Elliot <elliotfriedman3@gmail.com>
@ElliotFriedman ElliotFriedman removed the request for review from prat-gpt October 15, 2025 21:21
@ElliotFriedman
Copy link
Contributor Author

ElliotFriedman commented Oct 15, 2025

Putting this on the back burner for now since it will require a redeployment across all chains. #63

/// @dev minimum delay for timelocked operations
// minimum delay for timelocked operations
// there is no minimum timelock duration enforced when creating a timelock,
// however it is recommended to set a timelock duration of at least a few hours as an end user
Copy link
Contributor Author

Choose a reason for hiding this comment

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

remove this constant

Copy link
Contributor Author

Choose a reason for hiding this comment

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

maybe just move it into a test fixture file instead of leaving in constants

@ElliotFriedman
Copy link
Contributor Author

fix failing integration test: testValidateDeployment

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant