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

Duplicate algorithm implementations #785

Open
tatticoder opened this issue Oct 20, 2020 · 1 comment
Open

Duplicate algorithm implementations #785

tatticoder opened this issue Oct 20, 2020 · 1 comment

Comments

@tatticoder
Copy link
Contributor

I found 2 implementations for merge sort here and here.
Both are pretty similar in approach and solve same problem in same programming language.
One of them should be removed to reduce confusion and easier understanding.
I also suggest contributor should be notified for making a spam pull requests and Maintainers who merged requests without reviewing the code properly.

@sunilgknair051
Copy link
Contributor

sunilgknair051 commented Oct 20, 2020

@tatticoder , thanks for the heads up. I think on a rainy day there would have been a situation wherein the maintainers never looked at the content of the commit before merging. Our bad, sorry!

@sukritishah15, please do have a look.

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants