search

XRPLF / clio

An XRP Ledger API Server
https://xrpl.org
ISC License
57 stars 48 forks source link

Warn about deprecated fields #1314

Closed kuznetsss closed 2 months ago

kuznetsss commented 3 months ago

Fixes #1251.

codecov[bot] commented 3 months ago

Codecov Report

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

Project coverage is 64.38%. Comparing base (2302122) to head (6b995a1).

:exclamation: Current head 6b995a1 differs from pull request most recent head 9e125d5. Consider uploading reports for the commit 9e125d5 to get more accurate results

Files Patch % Lines
src/rpc/RPCEngine.hpp 0.00% 8 Missing :warning:
src/rpc/common/Types.hpp 60.00% 5 Missing and 1 partial :warning:
src/rpc/common/impl/Factories.hpp 50.00% 0 Missing and 6 partials :warning:
src/rpc/handlers/Ledger.hpp 0.00% 4 Missing :warning:
src/rpc/common/Checkers.hpp 82.35% 0 Missing and 3 partials :warning:
src/rpc/common/ValidationHelpers.hpp 82.35% 3 Missing :warning:
src/rpc/common/impl/Processors.hpp 57.14% 1 Missing and 2 partials :warning:
src/rpc/handlers/AccountInfo.hpp 0.00% 3 Missing :warning:
src/rpc/common/Specs.hpp 50.00% 0 Missing and 2 partials :warning:
src/rpc/common/impl/ForwardingProxy.hpp 33.33% 0 Missing and 2 partials :warning:
... and 7 more
Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #1314 +/- ## =========================================== - Coverage 64.46% 64.38% -0.08% =========================================== Files 246 248 +2 Lines 10460 10544 +84 Branches 5816 5839 +23 =========================================== + Hits 6743 6789 +46 - Misses 1899 1920 +21 - Partials 1818 1835 +17 ```

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