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

Fix for #315 (Aave v2 Avalanche) #318

Open
wants to merge 1 commit into
base: release/aave-v2-avalanche
Choose a base branch
from

Conversation

eboadom
Copy link
Collaborator

@eboadom eboadom commented Oct 20, 2022

Same exact changes as on #316 but to apply on the deployment of Aave v2 Avalanche.

Only non-solidity code was slightly adapted to the version of the repo from where this branch was created https://github.com/aave/protocol-v2/tree/Deployment%23001, the same version as on the target branch https://github.com/aave/protocol-v2/tree/release/aave-v2-avalanche

@eboadom
Copy link
Collaborator Author

eboadom commented Oct 20, 2022

Gas report diff

As expected, slight increase on average in single actions (~200 gas), with the benefit appearing on multiple actions in the same timestamp, same block.

Before fix
pre-100-rf-fix-gas-v2-avalanche

After fix
post-100-rf-fix-gas-v2-avalanche

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