Skip to content

Conversation

@ethanhaines
Copy link
Contributor

Hello again! I had some free time over the past few days and decided to add B-tree implementation, which I am familiar with but I wanted to get more practice with it. I made sure to add comprehensive tests that cover each method I added and ensured they all passed. I also made sure that the project built and ran without issue. Once again, let me know if there are any issues with my code, and I will work on it as soon as I can. Thank you for your time!

  • I have performed a self-review of my code
  • My code follows the style guidelines of this project
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Comments in areas I changed are up to date
  • I have added comments to hard-to-understand areas of my code
  • I have made corresponding changes to the README.md

@ethanhaines ethanhaines requested a review from siriak as a code owner November 3, 2025 22:43
@codecov
Copy link

codecov bot commented Nov 3, 2025

Codecov Report

❌ Patch coverage is 99.28401% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 96.81%. Comparing base (9d8d6d6) to head (f8217b2).
⚠️ Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
DataStructures/BTree/BTree.cs 99.24% 2 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #551      +/-   ##
==========================================
+ Coverage   96.72%   96.81%   +0.09%     
==========================================
  Files         282      284       +2     
  Lines       11174    11593     +419     
  Branches     1601     1668      +67     
==========================================
+ Hits        10808    11224     +416     
- Misses        232      234       +2     
- Partials      134      135       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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

@ethanhaines
Copy link
Contributor Author

I read the Codecov report and updated my tests to add coverage for the missing lines.

Copy link
Member

@siriak siriak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks!

@siriak siriak merged commit 8ec0781 into TheAlgorithms:master Nov 4, 2025
4 checks passed
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