codecov-commenter
commented
3 months ago Codecov Report
Attention: Patch coverage is 96.42857% with 5 lines in your changes are missing coverage. Please review.
Project coverage is 76.98%. Comparing base (
c88166e) to head (7e4a770).
| Files | Patch % | Lines |
|---|---|---|
| src/test/app/Flow_test.cpp | 95.83% | 4 Missing and 1 partial :warning: |
Additional details and impacted files
```diff @@ Coverage Diff @@ ## develop #4923 +/- ## =========================================== + Coverage 76.97% 76.98% +0.01% =========================================== Files 1129 1129 Lines 131914 132045 +131 Branches 39629 39726 +97 =========================================== + Hits 101539 101659 +120 - Misses 24459 24519 +60 + Partials 5916 5867 -49 ```:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
High Level Overview of Change
Changes the internal default path to include the path of the XRP bridge if sfPaths is unspecified in the Payment transaction.
Context of Change
https://github.com/XRPLF/XRPL-Standards/discussions/175
Type of Change
Before / After
If relevant, use this section for an English description of the change at a technical level. If no path is specified, the impact of the transaction on the node is increased, but if a path is set to mediate XRP, the impact on the node is unaffected.