search

codesankalp / dsalgo

MIT License
8 stars 24 forks source link

added merge() funtion and test cases #43

Open hemabhagnani opened 3 years ago

hemabhagnani commented 3 years ago

fixes #35

codesankalp commented 3 years ago

@hemabhagnani this pull request looks good to me and will be ready to merge after mentioned changes. Thanks!

hemabhagnani commented 3 years ago

@codesankalp I'll do the suggested changes

devkapilbansal commented 3 years ago

@hemabhagnani are you working on this?

hemabhagnani commented 3 years ago

Actually, I wanted to ask whether I have to change the approach of the merge() function apart from this I have done the requested changes.

devkapilbansal commented 3 years ago

@codesankalp @17sushmita review it