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

WIP meta pool contracts with gauge token support #773

Draft
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

penandlim
Copy link
Contributor

@penandlim penandlim commented Dec 2, 2022

Adds 4 new contracts

  • MetaSwapDepositGaugeSupportV1
  • MetaSwapGaugeSupportV1
  • MetaSwapUtilsGaugeSupportV1
  • GaugeTokenHolder

Meta* contracts are cloned from existing metaswap contracts with modifications to support depositing/withdrawing from gauges. MetaSwapGaugeSupportV1 has a new inheritence from GaugeTokenHolder.

@codecov
Copy link

codecov bot commented Dec 2, 2022

Codecov Report

Merging #773 (70ea0b3) into master (ccb4a6b) will decrease coverage by 15.24%.
The diff coverage is 0.00%.

❗ Current head 70ea0b3 differs from pull request most recent head e12c4e6. Consider uploading reports for the commit e12c4e6 to get more accurate results

@@             Coverage Diff             @@
##           master     #773       +/-   ##
===========================================
- Coverage   92.88%   77.64%   -15.24%     
===========================================
  Files          52       56        +4     
  Lines        3052     3651      +599     
  Branches      634      595       -39     
===========================================
  Hits         2835     2835               
- Misses        217      816      +599     
Impacted Files Coverage Δ
contracts/meta/MetaSwapDepositGaugeSupportV1.sol 0.00% <0.00%> (ø)
contracts/meta/MetaSwapGaugeSupportV1.sol 0.00% <0.00%> (ø)
contracts/meta/MetaSwapUtilsGaugeSupportV1.sol 0.00% <0.00%> (ø)
contracts/xchainGauges/GaugeTokenHolder.sol 0.00% <0.00%> (ø)
contracts/rewards/RewardScheduler.sol 0.00% <0.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@penandlim penandlim marked this pull request as draft December 2, 2022 08:02
@penandlim penandlim force-pushed the john/metaSwapWithGaugeSupport branch from 8ade918 to fdcc658 Compare December 6, 2022 06:50

// Check the last element of _pooledTokens is a gauge token
// This is a naiive check
IERC20 guageToken = _pooledTokens[_pooledTokens.length - 1];
Copy link
Contributor

@hannespfeiffer hannespfeiffer Dec 7, 2022

Choose a reason for hiding this comment

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

Nit: "gauge" misspelled

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