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 copy constructor and test case for min heap #47
base: main
Are you sure you want to change the base?
Conversation
I think there's a library missing for INT_MAX/INT_MIN cause the build is failing. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #47 +/- ##
==========================================
- Coverage 97.69% 97.50% -0.19%
==========================================
Files 85 91 +6
Lines 3036 3249 +213
Branches 494 526 +32
==========================================
+ Hits 2966 3168 +202
- Misses 70 81 +11
|
…e to T type into heapify function
I do not know what keeps failing. Everything seems to be alright on my machine. |
Can you add some if statements and maybe some more checks for the code? |
I think the add copy constructors to all classes issue can be closed now