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: update stuck keys #41

Merged
merged 10 commits into from
Dec 4, 2023
Merged

feat: update stuck keys #41

merged 10 commits into from
Dec 4, 2023

Conversation

skhomuti
Copy link
Contributor

now if NO has stuck keys, targetLimit will be applied to prevent further deposits

src/CSModule.sol Outdated Show resolved Hide resolved
src/lib/ValidatorCountsReport.sol Show resolved Hide resolved
src/lib/ValidatorCountsReport.sol Outdated Show resolved Hide resolved
src/CSModule.sol Outdated Show resolved Hide resolved
vgorkavenko
vgorkavenko previously approved these changes Nov 23, 2023
src/CSModule.sol Show resolved Hide resolved
src/CSModule.sol Outdated Show resolved Hide resolved
src/CSModule.sol Outdated Show resolved Hide resolved
src/CSModule.sol Outdated Show resolved Hide resolved
src/CSModule.sol Outdated Show resolved Hide resolved
src/CSModule.sol Outdated Show resolved Hide resolved
src/CSModule.sol Outdated Show resolved Hide resolved
src/CSModule.sol Outdated Show resolved Hide resolved
src/CSModule.sol Outdated Show resolved Hide resolved
test/CSModule.t.sol Outdated Show resolved Hide resolved
test/CSModule.t.sol Outdated Show resolved Hide resolved
src/CSModule.sol Outdated Show resolved Hide resolved
src/CSModule.sol Outdated Show resolved Hide resolved
src/CSModule.sol Outdated Show resolved Hide resolved
src/CSModule.sol Show resolved Hide resolved
src/CSModule.sol Outdated Show resolved Hide resolved
vgorkavenko
vgorkavenko previously approved these changes Dec 1, 2023
dgusakov
dgusakov previously approved these changes Dec 1, 2023
src/CSModule.sol Show resolved Hide resolved
test/CSModule.t.sol Show resolved Hide resolved
test/CSModule.t.sol Show resolved Hide resolved
test/CSModule.t.sol Show resolved Hide resolved
test/CSModule.t.sol Outdated Show resolved Hide resolved
test/CSModule.t.sol Outdated Show resolved Hide resolved
test/CSModule.t.sol Show resolved Hide resolved
test/CSModule.t.sol Show resolved Hide resolved
test/ValidatorCountsReport.t.sol Outdated Show resolved Hide resolved
test/ValidatorCountsReport.t.sol Show resolved Hide resolved
@skhomuti skhomuti dismissed stale reviews from dgusakov and vgorkavenko via 6c92415 December 4, 2023 06:39
madlabman
madlabman previously approved these changes Dec 4, 2023
vgorkavenko
vgorkavenko previously approved these changes Dec 4, 2023
now if NO has stuck keys, targetLimit will be applied to prevent further deposits
fix: reworked depositableValidatorsCount calculations
a lot of new tests
@skhomuti skhomuti merged commit a002653 into main Dec 4, 2023
4 checks passed
@skhomuti skhomuti deleted the stuck-keys branch December 4, 2023 08:15
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.

5 participants