Skip to content

add lightMigrator to ironblocks branch #44

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Dec 22, 2023
Merged

add lightMigrator to ironblocks branch #44

merged 4 commits into from
Dec 22, 2023

Conversation

YouStillAlive
Copy link
Member

No description provided.

@YouStillAlive
introduce LightMigrator (#39)
baa663e 
* introduce LightMigrator

* separate CreateNewPool tests

* fix codefactor

* use block timestamp

* codefactor fix
@YouStillAlive
extra tests (#40)
5e1bb74 
* extra tests

* codefactor fix

* name fix
@YouStillAlive
fix check (#42)
ae95bc4
@YouStillAlive
Merge branch 'ironblocks' into tmp-master
382e6ae
@github-actions GitHub Actions
Copy link

Gas Test Report 📊

The gas test results are as follows:

DelayVaultProvider

Contract Method Min Max Avg # calls
DelayVaultProvider createNewDelayVault(address,uint256[]) 305116 360961 335328 46
DelayVaultProvider createNewDelayVaultWithSignature(address,uint256[],bytes) - - 346548 1
DelayVaultProvider upgradeType(uint8) - - 72889 2

Old DelayVault

Contract Method Min Max Avg # calls
DelayVault CreateVault(address,uint256,uint256,uint256,uint256) 181746 266275 237177 12
DelayVault + DelayVaultMigrator Withdraw(address) 328314 443816 372507 8

DelayVaultMigrator

Contract Method Min Max Avg # calls
DelayVaultMigrator finalize(address) 78420 78432 78429 4
DelayVaultMigrator fullMigrate() 360650 367913 364282 2
DelayVaultMigrator withdrawTokensFromV1Vault() 348420 351220 349820 2

Deployments

Contract Min Max Avg % of limit
DelayVaultMigrator - - 1900650 1.5 %
DelayVaultProvider 4116713 4132372 4117190 3.2 %

@Lomet Lomet merged commit 3b6d275 into ironblocks Dec 22, 2023
@Lomet Lomet deleted the tmp-master branch December 22, 2023 13:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@YouStillAlive @Lomet