Skip to content

Randomize Quick Sort Addition #6234

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 12 commits into from
May 9, 2025

Conversation

VibhuKhera123
Copy link
Contributor

  • I have read CONTRIBUTING.md.
  • This pull request is all my own work -- I have not plagiarized it.
  • All filenames are in PascalCase.
  • All functions and variable names follow Java naming conventions.
  • All new algorithms have a URL in their comments that points to Wikipedia or other similar explanations.
  • All new code is formatted with clang-format -i --style=file path/to/your/file.java

@codecov-commenter
Copy link

codecov-commenter commented May 6, 2025

Codecov Report

Attention: Patch coverage is 91.30435% with 2 lines in your changes missing coverage. Please review.

Project coverage is 73.92%. Comparing base (6fe630c) to head (df355e9).
Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
.../thealgorithms/randomized/RandomizedQuickSort.java 91.30% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #6234      +/-   ##
============================================
- Coverage     73.92%   73.92%   -0.01%     
- Complexity     5327     5333       +6     
============================================
  Files           675      676       +1     
  Lines         18452    18475      +23     
  Branches       3572     3574       +2     
============================================
+ Hits          13640    13657      +17     
- Misses         4264     4268       +4     
- Partials        548      550       +2     

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

@VibhuKhera123 VibhuKhera123 marked this pull request as ready for review May 6, 2025 19:50
siriak
siriak previously approved these changes May 8, 2025
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
Copy link
Member

siriak commented May 8, 2025

@DenizAltunkapan please review again and merge if everything is good

Copy link
Collaborator

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

Very good. Thank you!

@siriak siriak enabled auto-merge (squash) May 9, 2025 21:46
@siriak siriak merged commit b09766e into TheAlgorithms:master May 9, 2025
6 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants