generated from Kwenta/foundry-scaffold
-
Notifications
You must be signed in to change notification settings - Fork 4
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
🚀 Octopus Release #54
Draft
JaredBorders
wants to merge
154
commits into
main
Choose a base branch
from
octopus
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* 🪲 SafeTransferFrom * ⛳️ remove try catch and fmt * 👷 added PlumberDesignated event * ✨ removed unused safe erc20 methods * ⛳ revert if an amount of zero is passed to _push and _pull --------- Co-authored-by: meb <[email protected]>
* 👷 SingleSwaps -> MultiHop WIP no tests * ✅ swapFor tests * ✨ added comment to prevent path gotcha * fixed quote fns * swap tests * fmt * 👷 added return vars back to quote fns * ✨ wasp review fixes * 👷 fixed IUniswap Quoter interface * 👷 fixed execute operation bytes decoding
* ✅ swap with tests * ✨ removed double factory declaration
* 👷 SingleSwaps -> MultiHop WIP no tests * ✅ swapFor tests * ✨ added comment to prevent path gotcha * fixed quote fns * swap tests * fmt * 👷 added return vars back to quote fns * ✨ wasp review fixes * 👷 fixed IUniswap Quoter interface * 👷 fixed execute operation bytes decoding * tolerance rename wip * ✅ testing multihop wip * ✨ fmt * ✅ fixed arbies multihop test * ✨ swap with tolerance rename * 📚 doc update * finished tolerance rename
* 👷 wip odos swap * forge install: surl * ⚙️ add ffi to foundry.toml * 👷 added odos router to zap constructor * ✅ successful odos quote using SURL * ✅ arb quote test and fmt * ✅ successfully calling odosAssemble * ✅ wip odos testing * ✨ Lint * ✅ blocked on testing * ✅ test failing in the odos swap * ✅ latest block on base fixed odos test * ✅ swap for tests + docs * ✅ asserts on swap for tests * ✅ fork tests take current block * ✨ removed all uniswap logic / mentions, renamed odos vars, updated docs * 👷 replaced ternary logic in unwind * ✨ removed unused constants * ✨ fmt * 📚 docs + fmt * ✨ renamed swapFrom test * fmt * ✅ added new bootstrap with current block forking * ✨ fmt * burn test * ✨fmt * 👷 fix: odos unwind swap return (#57) * 👷 fix: odos unwind swap return * fix * ✨ fmt --------- Co-authored-by: Moss <[email protected]> * 👷🏻♂️ Odos refund + Unwind Test (#58) * 👷 refund logic * ✅ unwind test * 👷 Odos audit fix (#59) * 📚 first round of fixes * 👷 fix 2 * some test fixes * 👷 fix leftovers (#60) * 👷 fix leftovers * fmt --------- Co-authored-by: Moss <[email protected]> * audit remidiations 23/11 * 👷 zapOut when unwinding USDC (#64) * 👷 zapOut when unwinding USDC * fix make deploy_base * 👷 adjust minAmountOut to match usdc 6 decimal precision when unwinding udsc * 👷 test fix * ✨ fmt
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.