Avoid mutating input array in bubble sort implementations by MayankSharma-2812 · Pull Request #1876 · TheAlgorithms/JavaScript
TheAlgorithms / JavaScript Public
-
-
Notifications
You must be signed in to change notification settings - Fork 5.9k
Open
wants to merge 5 commits into
Open
Avoid mutating input array in bubble sort implementations#1876
wants to merge 5 commits into
Avoid mutating input array in bubble sort implementations#1876
wants to merge 5 commits into
Conversation
MayankSharma-2812
commented
Jan 26, 2026
Ensures both bubble sort implementations return a sorted copy of the input array while preserving the original input.
MayankSharma-2812 added 5 commits
Codecov Report
❌ Patch coverage is 84.37500% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 85.87%. Comparing base (08d8c6b) to head (ddf6fb2).
| Files with missing lines | Patch % | Lines |
|---|---|---|
| Search/BinarySearch.js | 66.66% | 4 Missing |
| Sorts/BubbleSort.js | 82.60% | 4 Missing |
| Sorts/SelectionSort.js | 71.42% | 2 Missing |
Additional details and impacted files
@@ Coverage Diff @@ ## master #1876 +/- ## ========================================== - Coverage 85.91% 85.87% -0.05% ========================================== Files 379 379 Lines 19778 19789 +11 Branches 3015 3030 +15 ========================================== + Hits 16993 16994 +1 - Misses 2785 2795 +10
☔ 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.
- 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants
