-
Notifications
You must be signed in to change notification settings - Fork 28
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: addresses provider points to up to date deployment #53
Open
pmdaly
wants to merge
55
commits into
yearn:master
Choose a base branch
from
pmdaly:fix/yearn-provider-address-update
base: master
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.
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
…66] (yearn#6) * fix: modified prices helper to allow oracle to be upgraded * fix: removed unnecessary comma and newlines * fix: removed print * fix: removed newline
* feat: add synth calculations * fix: add tests and methods to update feeds * fix: cover oracle fallback logic with tests for synth assets * fix: use updated synth calcs fixture
…n#12) * feat: add synth calculations * fix: add tests and methods to update feeds * fix: cover oracle fallback logic with tests for synth assets * fix: use updated synth calcs fixture * fix: support curve lp tokens that are not in the curve registry * fix: add test for cvxCRV price * fix: make function internal
* fix: update curve calculations for new crypto pools * fix: correct logic for crypto pool price calculation and add ownable * fix: add address suffix
…yearn#18) * fix: attempt to fetch pool from crypto pool registry before main regstiry * chore: remove whitespace
* fix: detect crypto pool lp tokens * fix: fix test test_update_prices_helper_oracle_address
* fix: update user borrow limit on iron bank * contracts/RegistryAdapters/IronBankRegistryAdapter.sol: support cap * contracts/RegistryAdapters/IronBankRegistryAdapter.sol: fix by feedback Co-authored-by: bun919tw <[email protected]>
…earn#8) * feat: ib registry adapter beta fixes * chore: change code for consistancy across adapters * chore: revert few changes not needed any more
…arn#23) * test: added tests for the ironbank registry adapter * refactor: minor formatting changes
* first pass multicall for normalized price * struct as arg * simple tests for price helper
* price helper is now ownable * fixture updated
…earn#28) * feat: github ci, linting dependencies, readme.md added Signed-off-by: B3RX8OIPDHDT3 <[email protected]> * fix: remove ethlint
* Helper is now ownable * DelegatedBalanceMapping is now ownable * AddressesGeneratorV2Vaults is now ownable * AddressesGeneratorV1Vaults is now ownable * AddressGeneratorEarn is now ownable * AddressGeneratorEarn tests fixed * Oracle is now ownable * CalculationsSynth is now ownable * AddressesGeneratorIronBank is now ownable * GenericRegistry is now ownable * Manageable no longer needed * Adapter is now ownable & cleaning up * assetsStrategies is no longer used * moving fixtures to conf * removing deprecated tests and updating others Co-authored-by: x48 <[email protected]>
* feat: add chalink registry calculations * fix: rename variable
* managementList fixture revived * fixing feed addresses
* feat: linting performed on all .sol & .py files, workflows enabled * feat: .sol files linting * feat: add test to package.json * fix: indentation config + re-run linting * feat: use alchemy/custom explorer for brownie tests * fix: ganache cli env var * fix: tests run * fix: test option left out in brownie-config.yaml * fix: pr test pipeline disabled for now * fix: explorer env var expansion not available via config
* managementList fixture revived * fixing feed addresses * assetsStrategies properly called * cySushi now correctly has mint method * moving registry fixuture to conf * from_explorer will resolve correct methods * use registry to get latest vault address * IB & V2 registries and oracle addresses migrated * synth calc addresses moved to conftest * simpler access pattern * delegated balance -> use common address file * generators -> use common address file * helpers -> use common address file * oracle -> use common address file * registry adapetrs -> use common address file * tvl adapters -> use common address file * black linting and formatting * sorting imports
* feat: first pass override contract * refactor: very rough update * test: adding simple tests * feat: added staticcall * fix: all staticcall resp handeled correctly * fix: test linting * test: adding a check for pool list * refactor: removing staticcal to prefer calling known registries w/ pools * fix: added license and updated sol version * refactor: hard coded registries to owner supplied * refactor: hard coded registries to owner supplied * fix: curve addresses provider no longer needed * fix: override correctly instantiated * fix: returning solidity versions back to fixed
* refactor: more explicit address naming * fix: downstream expects 0x0 vs revert * feat: adding override to calculations curve * fix: removing rogue comment * fix: using new sushi whale address * feat: adding override to yearn address provider * feat: updating curve calculations contract to use override * fix: blocks per year test matches contract
…ty (yearn#49) * fix: route univ3 price calculations through weth to avoid low liquidity * fix: pass addresses in through constructor * chore: indentation * fix: add check for zero liquidity pools * feat: optimize gas cost with internal and no mstore
* refactor: yearn addresses provider added * fix: use a non usdc test for usdc normalization
* Updated getIronBankMarketPriceUsdc price calculation * feat: Add iron bank tests for cUsdc, cDai and cyUsdc
jstashh
approved these changes
May 4, 2022
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.
Previous yearn addressses provider address was bricked. Deployment address updated.