Skip to content

Conversation

UdayaKrishnanM
Copy link
Contributor

@UdayaKrishnanM UdayaKrishnanM commented Oct 4, 2025

Hi @DenizAltunkapan as discussed last week for this PR I have attached the pretty display method. Can you review and let me know for any changes. Also the BSTRecursiveGenericTest file is missing also added it

@codecov-commenter
Copy link

codecov-commenter commented Oct 4, 2025

Codecov Report

❌ Patch coverage is 44.44444% with 15 lines in your changes missing coverage. Please review.
✅ Project coverage is 76.29%. Comparing base (959ced9) to head (0aaa73b).

Files with missing lines Patch % Lines
...thms/datastructures/trees/BSTRecursiveGeneric.java 44.44% 15 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #6621      +/-   ##
============================================
+ Coverage     76.06%   76.29%   +0.22%     
- Complexity     5885     5913      +28     
============================================
  Files           709      709              
  Lines         20085    20111      +26     
  Branches       3897     3899       +2     
============================================
+ Hits          15278    15343      +65     
+ Misses         4219     4177      -42     
- Partials        588      591       +3     

☔ 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.

@UdayaKrishnanM UdayaKrishnanM changed the title Added Pretty Display method and updated Tests #6548 Added Pretty Display method and BSTRecursivceGenericTests #6548 Oct 4, 2025
@UdayaKrishnanM UdayaKrishnanM changed the title Added Pretty Display method and BSTRecursivceGenericTests #6548 Added Pretty Display method and BSTRecursivceGenericTests.java Oct 4, 2025
@UdayaKrishnanM UdayaKrishnanM changed the title Added Pretty Display method and BSTRecursivceGenericTests.java Added Pretty Display method and BSTRecursiveGenericTests.java Oct 4, 2025
@UdayaKrishnanM
Copy link
Contributor Author

Hi @DenizAltunkapan @alxkm @yanglbme . Can anyone check my PR

@UdayaKrishnanM
Copy link
Contributor Author

Hi @alxkm , please check out this PR

Copy link
Member

@DenizAltunkapan DenizAltunkapan 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!

@DenizAltunkapan DenizAltunkapan enabled auto-merge (squash) October 8, 2025 11:53
@DenizAltunkapan DenizAltunkapan merged commit 5f8d8ee into TheAlgorithms:master Oct 8, 2025
6 checks passed
@UdayaKrishnanM
Copy link
Contributor Author

thank you very much @DenizAltunkapan

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.

3 participants