Skip to content
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

Iss20 #47

Merged
merged 3 commits into from
Feb 5, 2017
Merged

Iss20 #47

merged 3 commits into from
Feb 5, 2017

Conversation

nbro
Copy link
Owner

@nbro nbro commented Feb 5, 2017

No description provided.

- module Heap.py renamed to heap.py, since it could contain more than
one class representing heaps
- Removed HeapPrinter class, which had only one method, and made this
method a standalone function
- Removed class Heap, from which MinHeap and MaxHeap derived, since it
only contain one method, “delete”, which was moved to its child classes.
- Refactored a few things around
@coveralls
Copy link

coveralls commented Feb 5, 2017

Coverage Status

Coverage increased (+0.2%) to 94.259% when pulling f0f80f7 on iss20 into c40c800 on master.

@nbro nbro merged commit 64335f5 into master Feb 5, 2017
@nbro nbro deleted the iss20 branch February 5, 2017 02:33
Sign up for free to 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.

None yet

2 participants