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

Feature/lend refactor #349

Merged
merged 12 commits into from
Jul 21, 2022
Merged

Feature/lend refactor #349

merged 12 commits into from
Jul 21, 2022

Conversation

pratikasr
Copy link
Member

No description provided.

}
balanceStats = append(balanceStats, balanceStat)
depositStats = types.DepositStats{BalanceStats: balanceStats}
userDepositStats = types.DepositStats{BalanceStats: balanceStats}

Check warning

Code scanning / CodeQL

Useless assignment to local variable

This definition of userDepositStats is never used.
balanceStats = append(balanceStats, balanceStat)
depositStats = types.DepositStats{BalanceStats: balanceStats}
userDepositStats = types.DepositStats{BalanceStats: balanceStats}
ReserveDepositStats = types.DepositStats{BalanceStats: balanceStats}

Check warning

Code scanning / CodeQL

Useless assignment to local variable

This definition of ReserveDepositStats is never used.
depositStats = types.DepositStats{BalanceStats: balanceStats}
userDepositStats = types.DepositStats{BalanceStats: balanceStats}
ReserveDepositStats = types.DepositStats{BalanceStats: balanceStats}
BuyBackDepositStats = types.DepositStats{BalanceStats: balanceStats}

Check warning

Code scanning / CodeQL

Useless assignment to local variable

This definition of BuyBackDepositStats is never used.
userDepositStats = types.DepositStats{BalanceStats: balanceStats}
ReserveDepositStats = types.DepositStats{BalanceStats: balanceStats}
BuyBackDepositStats = types.DepositStats{BalanceStats: balanceStats}
BorrowStats = types.DepositStats{BalanceStats: balanceStats}

Check warning

Code scanning / CodeQL

Useless assignment to local variable

This definition of BorrowStats is never used.
x/liquidation/abci.go Fixed Show fixed Hide fixed
x/liquidation/abci.go Fixed Show fixed Hide fixed
x/liquidation/abci.go Fixed Show fixed Hide fixed
x/esm/abci.go Outdated
esmMarket, found := k.GetESMMarketForAsset(ctx, v.Id)
if found {
return
}
esmStatus, found := k.GetESMStatus(ctx, v.Id)

Check warning

Code scanning / CodeQL

Panic in BeginBock or EndBlock consensus methods

Possible panics in BeginBock- or EndBlock-related consensus methods could cause a chain halt
if err != nil {
return
}
err = k.LiquidateBorrows(ctx)

Check warning

Code scanning / CodeQL

Panic in BeginBock or EndBlock consensus methods

Possible panics in BeginBock- or EndBlock-related consensus methods could cause a chain halt
if err != nil {
return
}
err = k.UpdateLockedBorrows(ctx)

Check warning

Code scanning / CodeQL

Panic in BeginBock or EndBlock consensus methods

Possible panics in BeginBock- or EndBlock-related consensus methods could cause a chain halt
x/auction/abci.go Fixed Show fixed Hide fixed
if err4 != nil {
return
}
err5 := k.LendDutchActivator(ctx)

Check warning

Code scanning / CodeQL

Panic in BeginBock or EndBlock consensus methods

Possible panics in BeginBock- or EndBlock-related consensus methods could cause a chain halt
if err5 != nil {
return
}
err6 := k.RestartLendDutch(ctx)

Check warning

Code scanning / CodeQL

Panic in BeginBock or EndBlock consensus methods

Possible panics in BeginBock- or EndBlock-related consensus methods could cause a chain halt
@pratikasr pratikasr marked this pull request as ready for review July 20, 2022 17:24
@dheerajkd30 dheerajkd30 merged commit 46135c2 into feature/dev Jul 21, 2022
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