-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Refresh Pancake Sort #64
Conversation
Codecov Report
@@ Coverage Diff @@
## master #64 +/- ##
==========================================
+ Coverage 17.8% 19.98% +2.18%
==========================================
Files 32 32
Lines 1685 1671 -14
Branches 262 261 -1
==========================================
+ Hits 300 334 +34
+ Misses 1378 1330 -48
Partials 7 7
Continue to review full report at Codecov.
|
I will modify the PancakeSorter.cs file in line with codacy-bot's recommendation: Codacy Issue found: Do not update the loop counter 'mi' within the loop body. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Update it like that please, will fix Codacy
Reopened Pull Request |
Co-Authored-By: Andrii Siriak <siryaka@gmail.com>
Co-Authored-By: Andrii Siriak <siryaka@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool! Thank you for your contribution!
* Refresh Pancake Sort * Update Algorithms/Sorters/PancakeSorter.cs Co-Authored-By: Andrii Siriak <siryaka@gmail.com>
Refreshed the "Pancake Sorter" Implementation in line with other "sorter" refreshes.
One File Deletion
--> pancake_sort.cs
Two File Additions
--> Algorithms.Tests/Sorters/PancakeSorterTests.cs
--> Algorithms/Sorters/PancakeSorter.cs
Tested with Random Integer Array
Built successfully in Debug and Release