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

refactor obtainer to use normalized odds from storage also resize contract #290

Merged
merged 1 commit into from
Mar 3, 2023

Conversation

grujawork
Copy link
Contributor

No description provided.

@grujawork grujawork self-assigned this Mar 3, 2023
@codecov
Copy link

codecov bot commented Mar 3, 2023

Codecov Report

Patch coverage: 78.57% and project coverage change: -0.05 ⚠️

Comparison is base (1f91f5b) 85.04% compared to head (8433e4c) 85.00%.

Additional details and impacted files
@@                  Coverage Diff                   @@
##           amm-liquidity-pool     #290      +/-   ##
======================================================
- Coverage               85.04%   85.00%   -0.05%     
======================================================
  Files                      74       74              
  Lines                    2207     2200       -7     
  Branches                 1125     1114      -11     
======================================================
- Hits                     1877     1870       -7     
  Misses                    330      330              
Impacted Files Coverage Δ
...tracts/SportMarkets/Rundown/TherundownConsumer.sol 99.00% <ø> (ø)
...ntracts/SportMarkets/Rundown/GamesOddsObtainer.sol 89.06% <78.57%> (-1.08%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@danijelthales danijelthales merged commit 557fee4 into amm-liquidity-pool Mar 3, 2023
@grujawork grujawork deleted the normalized_odds_change branch March 3, 2023 16:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants