-
Notifications
You must be signed in to change notification settings - Fork 5
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: cow amm safe module #33
Conversation
7e4039b
to
bf254ba
Compare
bf254ba
to
8f0ffe8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mostly high-level comments on the current design.
Overall, I think it's a big improvement over the current deployment mechanism in the docs and we should recommend it. There are a few relatively small changes I'd like to see, however.
75906f5
to
30ff5d4
Compare
Ready for review, test suite coming in a subsequent PR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
There are some minor suggestion, but I'm not sure they are worth a redeployment on mainnet.
Co-authored-by: Federico Giacon <[email protected]>
This PR: