-
Notifications
You must be signed in to change notification settings - Fork 267
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
Issue #49 circular linked list #115
Issue #49 circular linked list #115
Conversation
Codecov Report
@@ Coverage Diff @@
## master #115 +/- ##
============================================
+ Coverage 97.94% 98.019% +0.078%
============================================
Files 20 20
Lines 1311 1363 +52
============================================
+ Hits 1284 1336 +52
Misses 27 27
|
Please resolve the conflicts. |
Resolved. |
f0df79b
to
7bb4305
Compare
========== | ||
|
||
.. [1] https://en.wikipedia.org/wiki/Linked_list#Circular_linked_list | ||
.. [2] https://www.geeksforgeeks.org/circular-singly-linked-list-insertion/ |
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.
.. [2] https://www.geeksforgeeks.org/circular-singly-linked-list-insertion/ |
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.
The wiki page does not have enough info so I added other references.
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.
The info on the Wiki is not enough. Should I add a reference from other sites or leave it as it is?
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.
You can add links to university lectures.
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.
I have updated Wiki References.
@prshnt19 If you want, you can add your name to AUTHORS file in a separate PR. |
LGTM. |
Co-Authored-By: Gagandeep Singh <[email protected]>
Tests are correct. I have verified it. |
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.
tests look okay to me
Added SinglyCircularLinkedList and DoublyCircularLinkedList
References to other Issues or PRs or Relevant literature
#49
Brief description of what is fixed or changed
Other comments