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

Pref/optimise migration #146

Merged
merged 2 commits into from
Apr 11, 2022
Merged

Conversation

parv3213
Copy link
Member

@parv3213 parv3213 commented Apr 8, 2022

This approximately saves a total of 26501 gas on calls and 76968 gas on deployment.

@parv3213 parv3213 requested a review from pegahcarter April 8, 2022 12:11
@parv3213 parv3213 self-assigned this Apr 8, 2022
Copy link
Collaborator

@zgorizzo69 zgorizzo69 left a comment

Choose a reason for hiding this comment

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

nice refactoring much appreciated ! 🥇

@pegahcarter pegahcarter merged commit f39405a into fix/remove-soonest Apr 11, 2022
@pegahcarter pegahcarter deleted the pref/optimise-migration branch April 11, 2022 15:13
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.

3 participants