xline-kv / Xline

A geo-distributed KV store for metadata management
https://xline.cloud
Apache License 2.0
596 stars 74 forks source link

refactor: enable new sp/ucp implementation #668

Closed bsbds closed 4 months ago

bsbds commented 7 months ago

Depends-On: #769

Please briefly answer these questions:

codecov[bot] commented 7 months ago

Codecov Report

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

Project coverage is 75.60%. Comparing base (e35b35a) to head (29695b3). Report is 70 commits behind head on master.

Files Patch % Lines
crates/curp-test-utils/src/test_cmd.rs 0.00% 5 Missing :warning:
crates/xlineapi/src/lib.rs 0.00% 5 Missing :warning:
crates/xlineapi/src/command.rs 0.00% 3 Missing :warning:
crates/curp/src/server/raw_curp/mod.rs 96.00% 2 Missing :warning:
crates/curp/src/server/metrics.rs 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #668 +/- ## ========================================== + Coverage 75.55% 75.60% +0.04% ========================================== Files 180 186 +6 Lines 26938 27472 +534 Branches 26938 27472 +534 ========================================== + Hits 20353 20770 +417 - Misses 5366 5421 +55 - Partials 1219 1281 +62 ```

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

mergify[bot] commented 5 months ago

@bsbds Convert your pr to draft since CI failed

mergify[bot] commented 5 months ago

@bsbds Your PR is in conflict and cannot be merged.