Skip to content

chore: improve ModuleBalance#959

Merged
toteki merged 5 commits intomainfrom
adam/mb
May 26, 2022
Merged

chore: improve ModuleBalance#959
toteki merged 5 commits intomainfrom
adam/mb

Conversation

@toteki
Copy link
Copy Markdown
Contributor

@toteki toteki commented May 26, 2022

Description

Tiny audit issue fix

closes: #896
closes: #868

@toteki toteki requested a review from a team as a code owner May 26, 2022 03:21
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #959 (8c613c9) into main (2bba354) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #959   +/-   ##
=======================================
  Coverage   50.90%   50.90%           
=======================================
  Files          64       64           
  Lines        9479     9479           
=======================================
  Hits         4825     4825           
  Misses       4401     4401           
  Partials      253      253           
Impacted Files Coverage Δ
x/leverage/keeper/keeper.go 45.88% <100.00%> (ø)

@adamewozniak
Copy link
Copy Markdown
Collaborator

Changelog pls :^) almost forgot

@toteki toteki requested a review from a team as a code owner May 26, 2022 16:23
@toteki toteki merged commit 784b508 into main May 26, 2022
@toteki toteki deleted the adam/mb branch May 26, 2022 17:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

x/leverage: switch ModuleBalance function to use SpendableCoins ( RV #25 ) x/leverage: Redundant ModuleBalance logic ( RV #16 )

3 participants