Skip to content

Conversation

@m-maksyutin
Copy link
Contributor

Hi, this is fixing the situation when we have an unsorted array with negative numbers and zeroes.

@codecov-io
Copy link

Codecov Report

Merging #86 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@ Coverage Diff @@## master #86 +/- ## ===================================== Coverage 100% 100% ===================================== Files 100 100 Lines 2029 2029 Branches 354 354 ===================================== Hits 2029 2029
Impacted FilesCoverage Δ
...algorithms/sorting/insertion-sort/InsertionSort.js100% <ø> (ø)⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e3b482c...96efc15. Read the comment docs.

@trekhleb
Copy link
Owner

Thank you for the fix @m-maksyutin

@trekhlebtrekhleb merged commit d82958d into trekhleb:masterJul 3, 2018
shoredata pushed a commit to shoredata/javascript-algorithms that referenced this pull request Mar 28, 2019
* Fix LinkedList * Fix the prepend method for the LinkedList * Fix the remove method for the MinHeap * Correct a comment * Fix BST removal method * Fix the findEdge method of the graph * Fix the value returned by DisjointSet union * Add recursive factorial function * Fix the insertion sort
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@m-maksyutin@codecov-io@trekhleb@odykyi@hdcorder147@MikhailMaksyutin