-
Notifications
You must be signed in to change notification settings - Fork 273
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
Added Binary Heap #46
Conversation
Added tests for Min and Max Heap. Added handling of extraction on a Empty Heap
ping @23umesh Apologies for closing your PR by mistake. Your commits are still there and your name will be added to authors file(only if you want). Let me know. |
Codecov Report
@@ Coverage Diff @@
## master #46 +/- ##
=============================================
+ Coverage 97.895% 98.003% +0.107%
=============================================
Files 15 17 +2
Lines 998 1102 +104
=============================================
+ Hits 977 1080 +103
- Misses 21 22 +1
|
@23umesh As you given thumbs up to above comment, I am adding your name to the AUTHORS file. Please let me know your email-id in the comment below. |
For now I have added what was available on your Github profile. Make a PR if you want any changes. For now I will merge as soon as the tests pass. Thanks for the contribution. |
References to other Issues or PRs or Relevant literature
#42
Brief description of what is fixed or changed
Other comments