Closed wthrajat closed 2 days ago
Attention: Patch coverage is 58.49057%
with 22 lines
in your changes missing coverage. Please review.
Project coverage is 74.80%. Comparing base (
2f9f0aa
) to head (0bbacec
). Report is 9 commits behind head on master.
Files with missing lines | Patch % | Lines |
---|---|---|
src/taker/api.rs | 48.00% | 13 Missing :warning: |
src/maker/server.rs | 44.44% | 5 Missing :warning: |
src/taker/routines.rs | 20.00% | 4 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Aims to fix https://github.com/citadel-tech/coinswap/issues/248
MakerConfig
andTakerConfig
that should not be exposed to users.maker/server.rs
.taker/api.rs
.directory_server_onion_address
anddirectory_server_clearnet_address
fields with a unifieddirectory_server_address
field for bothMakerConfig
andTakerConfig
.required_confirms
to a default value of1
forMakerConfig
, removing it from user-facing configuration options.