White-Whale-Defi-Platform / white-whale-core

Core contracts for White Whale, The Interchain Liquidity Protocol for the Cosmos.
https://whitewhale.money
MIT License
36 stars 23 forks source link

refactor: pool manager cleanup #350

Closed kerber0x closed 2 months ago

kerber0x commented 2 months ago

Description and Motivation

This does some refactoring, general cleanup, adding docs and so on.

Related Issues


Checklist:

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 94.57901% with 47 lines in your changes are missing coverage. Please review.

Project coverage is 90.55%. Comparing base (0f77427) to head (5e487cc). Report is 1 commits behind head on release/v2_contracts.

Files Patch % Lines
...ontracts/liquidity_hub/pool-manager/src/helpers.rs 70.93% 25 Missing :warning:
...ontracts/liquidity_hub/pool-manager/src/queries.rs 87.03% 7 Missing :warning:
...quidity_hub/pool-manager/src/liquidity/commands.rs 91.42% 3 Missing :warning:
...acts/liquidity_hub/pool-manager/src/tests/suite.rs 89.65% 3 Missing :warning:
...ty_hub/pool-manager/src/tests/integration_tests.rs 99.60% 2 Missing :warning:
...ntracts/liquidity_hub/pool-manager/src/contract.rs 97.05% 1 Missing :warning:
...liquidity_hub/pool-manager/src/manager/commands.rs 95.00% 1 Missing :warning:
contracts/liquidity_hub/pool-manager/src/math.rs 0.00% 1 Missing :warning:
.../liquidity_hub/pool-manager/src/router/commands.rs 95.45% 1 Missing :warning:
...ts/liquidity_hub/pool-manager/src/swap/commands.rs 96.42% 1 Missing :warning:
... and 2 more
Additional details and impacted files ```diff @@ Coverage Diff @@ ## release/v2_contracts #350 +/- ## ======================================================== + Coverage 90.09% 90.55% +0.45% ======================================================== Files 278 276 -2 Lines 29604 29921 +317 ======================================================== + Hits 26673 27095 +422 + Misses 2931 2826 -105 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.