Added insert_start function for Doubly Linked List

dennis-pg commented 3 years ago

Description

Addressing Issue #77 Added required insert_start function

Related Issue

Please don't create a Pull request without creating an issue first

Fixes #77

Type of Change:

Code

How Has This Been Tested?

Sanity checks ran

Checklist:

[x] I read the CONTRIBUTING document before making this pull request
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code or materials
[x] I have commented my code or provided relevant documentation, particularly in hard-to-understand areas
[ ] My change requires change to documentation
[ ] I have updated documentation accordingly
[ ] I have updated the requirements file
[x] My changes generate no new warnings
[ ] My PR currently breaks something (fix or feature that would cause existing functionality to not work as expected)
[ ] I have added tests to cover my changes
[ ] All new and existing tests passed

Screenshots (if appropriate):

devkapilbansal commented 3 years ago

@dennis-pg add test cases for this too

devkapilbansal commented 3 years ago

Also, @dennis-pg there are some more issues open related to it which are not assigned. Solve those too.

dennis-pg commented 3 years ago

Okay, will have a look

keshavgbpecdelhi commented 3 years ago

@dennis-pg, I have made some changes in your code. Feel free to squash them if you want

codesankalp commented 3 years ago

@dennis-pg are you still working on this issue?

dennis-pg commented 3 years ago

Sorry, got a bit caught up. Will finish up on this today

dennis-pg commented 3 years ago

Could you mention the issue numbers. Don't see any unresolved issues pertaining to doubly linked lists

dennis-pg commented 3 years ago

Have made the required changes

codesankalp commented 3 years ago

Please squash your commits. I will review your code after that @dennis-pg

dennis-pg commented 3 years ago

Squashed

codesankalp commented 3 years ago

@dennis-pg I see there are many commits please make them into one (squash in one commit).

dennis-pg commented 3 years ago

Done

devkapilbansal commented 3 years ago

@dennis-pg I will review and test your PR soon. Till then please follow the commit guidelines mentioned in the contribution guide.

codesankalp commented 3 years ago

Also changed the styling because at someplace it is in the wrong format. Tested OK.

Merging.

codesankalp / dsalgo