-
Notifications
You must be signed in to change notification settings - Fork 20k
refactor: SelectionSort
like classes and their tests
#5265
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
… too big. To avoid stackoverflow
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #5265 +/- ##
============================================
+ Coverage 39.95% 39.96% +0.01%
Complexity 2458 2458
============================================
Files 519 519
Lines 15501 15487 -14
Branches 2965 2957 -8
============================================
- Hits 6193 6190 -3
+ Misses 9017 9007 -10
+ Partials 291 290 -1 ☔ View full report in Codecov by Sentry. |
In this pull request, also changed SortingAlgorithmTest to customize size of generated array. For recursive selection sort, time to time this was an stackOverFlow, to avoid failed builds, size of generated arrray can be changed by demand. |
Hello @vil02 This PR should fix this problem https://github.com/TheAlgorithms/Java/actions/runs/9708998343/job/26796827946 I already runned this bunch of times and no such cases was. |
src/main/java/com/thealgorithms/sorts/SelectionSortRecursive.java
Outdated
Show resolved
Hide resolved
SelectionSort
like classes and their tests
Refactor: Adding test common approach, adding javadocs, renaming variables
clang-format -i --style=file path/to/your/file.java