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/univ3 multisig #204

Open
wants to merge 37 commits into
base: master
Choose a base branch
from
Open

Feat/univ3 multisig #204

wants to merge 37 commits into from

Conversation

MisterMard
Copy link
Collaborator

No description provided.

_cache.amount1
);
}
bool _zeroForOne = _cache.amount0Desired > _cache.amount0 ? true : false;
Copy link
Contributor

Choose a reason for hiding this comment

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

I know you inherited this naming convention, but it's so confusing that amount0Desired is what you already have, and amount0 is what you desire to obtain 😅

move polygon univ3 migration test to polygon dir
add tests for arbitrum univ3 migrations

chore: cleanup obsolete files

update safe-core-sdk to latest version to support safes on arbitrum
add tests for optimism univ3 migrations

chore: cleanup obsolete files
add tests for mainnet univ3 migrations

fix: multisigHelper ensures owner balance covers txn const
add tests for polygon univ3 migrations
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.

2 participants