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

feat: drop slashing reporting #370

Merged
merged 9 commits into from
Jan 7, 2025
Merged

feat: drop slashing reporting #370

merged 9 commits into from
Jan 7, 2025

Conversation

madlabman
Copy link
Contributor

@madlabman madlabman commented Dec 2, 2024

Pectra hard fork will bring a significant reduction in the initial slashing penalty, so it no longer makes sense to keep the method that delivers initial slashings to the module.

@madlabman madlabman force-pushed the verifier-drop-slashing branch from bdff0b3 to b552865 Compare December 4, 2024 11:16
skhomuti
skhomuti previously approved these changes Dec 5, 2024
src/CSModule.sol Outdated Show resolved Hide resolved
@madlabman madlabman force-pushed the verifier-drop-slashing branch from 33aab38 to dded936 Compare December 10, 2024 11:03
@madlabman madlabman marked this pull request as ready for review December 10, 2024 11:03
@madlabman madlabman requested a review from a team as a code owner December 10, 2024 11:03
script/DeployCSVerifier.s.sol Outdated Show resolved Hide resolved
script/DeployCSVerifier.s.sol Outdated Show resolved Hide resolved
script/DeployCSVerifier.s.sol Outdated Show resolved Hide resolved
@madlabman madlabman force-pushed the verifier-drop-slashing branch from 400b5a7 to af26445 Compare December 11, 2024 12:24
@dgusakov dgusakov merged commit 72eace1 into develop Jan 7, 2025
6 checks passed
@dgusakov dgusakov deleted the verifier-drop-slashing branch January 7, 2025 09:50
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.

4 participants