Skip to content

Conversation

@pratikmankawde
Copy link
Collaborator

Details to come

Signed-off-by: Pratik Mankawde <3397372+pratikmankawde@users.noreply.github.com>
Signed-off-by: Pratik Mankawde <3397372+pratikmankawde@users.noreply.github.com>
Signed-off-by: Pratik Mankawde <3397372+pratikmankawde@users.noreply.github.com>
Signed-off-by: Pratik Mankawde <3397372+pratikmankawde@users.noreply.github.com>
Signed-off-by: Pratik Mankawde <3397372+pratikmankawde@users.noreply.github.com>
Signed-off-by: Pratik Mankawde <3397372+pratikmankawde@users.noreply.github.com>
Signed-off-by: Pratik Mankawde <3397372+pratikmankawde@users.noreply.github.com>
Signed-off-by: Pratik Mankawde <3397372+pratikmankawde@users.noreply.github.com>
Signed-off-by: Pratik Mankawde <3397372+pratikmankawde@users.noreply.github.com>
Signed-off-by: Pratik Mankawde <3397372+pratikmankawde@users.noreply.github.com>
@pratikmankawde pratikmankawde requested a review from a team as a code owner December 17, 2025 14:04
@pratikmankawde pratikmankawde force-pushed the pratik/Add-sanitizers-to-CI-builds branch from 4a75da8 to 6479ab2 Compare December 18, 2025 15:46
…an-flagged-issues

Signed-off-by: Pratik Mankawde <3397372+pratikmankawde@users.noreply.github.com>
@codecov
Copy link

codecov bot commented Dec 19, 2025

Codecov Report

❌ Patch coverage is 95.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 79.1%. Comparing base (0ab3e11) to head (1aebb3f).
⚠️ Report is 1 commits behind head on pratik/Add-sanitizers-to-CI-builds.

Files with missing lines Patch % Lines
src/xrpld/peerfinder/detail/Counts.h 90.0% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@                        Coverage Diff                         @@
##           pratik/Add-sanitizers-to-CI-builds   #6151   +/-   ##
==================================================================
  Coverage                                79.1%   79.1%           
==================================================================
  Files                                     836     836           
  Lines                                   71243   71250    +7     
  Branches                                 8320    8319    -1     
==================================================================
+ Hits                                    56350   56358    +8     
+ Misses                                  14893   14892    -1     
Files with missing lines Coverage Δ
include/xrpl/basics/DecayingSample.h 88.9% <100.0%> (+0.3%) ⬆️
include/xrpl/nodestore/detail/varint.h 85.4% <100.0%> (+0.4%) ⬆️
include/xrpl/protocol/nft.h 100.0% <100.0%> (ø)
src/libxrpl/protocol/BuildInfo.cpp 98.1% <ø> (ø)
src/xrpld/app/main/BasicApp.cpp 100.0% <100.0%> (ø)
src/xrpld/peerfinder/detail/Counts.h 72.7% <90.0%> (+1.3%) ⬆️

... and 4 files with indirect coverage changes

Impacted file tree graph

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Signed-off-by: Pratik Mankawde <3397372+pratikmankawde@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants