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

feat: special asset pairs full implementation #2178

Merged
merged 82 commits into from
Aug 11, 2023
Merged

feat: special asset pairs full implementation #2178

merged 82 commits into from
Aug 11, 2023

Conversation

toteki
Copy link
Member

@toteki toteki commented Aug 2, 2023

See TODO comments in code

@codecov
Copy link

codecov bot commented Aug 2, 2023

Codecov Report

Merging #2178 (9599cbb) into main (7f05ad4) will decrease coverage by 6.21%.
Report is 179 commits behind head on main.
The diff coverage is 79.35%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2178      +/-   ##
==========================================
- Coverage   75.38%   69.18%   -6.21%     
==========================================
  Files         100      166      +66     
  Lines        8025    12551    +4526     
==========================================
+ Hits         6050     8683    +2633     
- Misses       1589     3316    +1727     
- Partials      386      552     +166     
Files Changed Coverage Δ
ante/spam_prevention.go 75.92% <ø> (ø)
cmd/ibc_denom/main.go 0.00% <0.00%> (ø)
util/coin/coin.go 12.50% <0.00%> (ø)
x/incentive/codec.go 47.82% <ø> (+9.89%) ⬆️
x/incentive/keeper/invariants.go 0.00% <0.00%> (ø)
x/incentive/keeper/reward.go 82.35% <ø> (ø)
x/incentive/keeper/store.go 70.67% <ø> (ø)
x/incentive/keeper/unbonding.go 80.45% <ø> (ø)
x/incentive/keeper/update.go 52.11% <ø> (ø)
x/incentive/keys.go 100.00% <ø> (ø)
... and 59 more

... and 67 files with indirect coverage changes

@toteki toteki changed the title feat: implement special asset pairs 2 feat: special asset pairs helpers Aug 2, 2023
@toteki toteki changed the title feat: special asset pairs helpers feat: special asset pairs full implementation Aug 8, 2023
@toteki toteki marked this pull request as ready for review August 8, 2023 20:53
@toteki toteki requested review from a team as code owners August 8, 2023 20:53
// also check collateral value vs weighted borrow (borrow factor limit)
otherCollateralValue, err := k.VisibleCollateralValue(ctx, otherCollateral, types.PriceModeLow)
if err != nil {
maxWithdraw := coin.Zero(uDenom)

Check warning

Code scanning / CodeQL

Useless assignment to local variable Warning

This definition of maxWithdraw is never used.
Copy link
Collaborator

@gsk967 gsk967 left a comment

Choose a reason for hiding this comment

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

I have given my approval and left some comments. Please have someone review this pull request as an extra precaution.

@gsk967 gsk967 enabled auto-merge August 11, 2023 14:52
@gsk967 gsk967 added this pull request to the merge queue Aug 11, 2023
Merged via the queue into main with commit 52106f7 Aug 11, 2023
26 checks passed
@gsk967 gsk967 deleted the adam/special4 branch August 11, 2023 15:35
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.

3 participants