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
3 participants