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

Insertion Sort script and documentation #5

Closed
Zircoz opened this issue Oct 9, 2019 · 5 comments

Comments

@Zircoz
Copy link
Owner

commented Oct 9, 2019

No description provided.

@akhilkh2000

This comment has been minimized.

Copy link

commented Oct 10, 2019

Can i take up this issue?

@Zircoz

This comment has been minimized.

Copy link
Owner Author

commented Oct 11, 2019

Hey @akhilkh2000 , @eric3119 already submitted a PR for this issue.
So, if you can improve upon his code/documentation please do it. Also, preferably explain how your (proposed?) changes improve the existing version.

@Zircoz

This comment has been minimized.

Copy link
Owner Author

commented Oct 11, 2019

@24sharkS You didn't mention this issue or me anywhere on your PR ( #11 ). In conditions like these, I may not know that you created a PR (considering there are so many of them).

Plus, Eric already got his commits merged, so now an improvement will be much appreciated.

@24sharkS

This comment has been minimized.

Copy link

commented Oct 12, 2019

@Zircoz, I would surely like to work on improving it.

@Zircoz

This comment has been minimized.

Copy link
Owner Author

commented Oct 12, 2019

Sure, @24sharkS Create a new issue when you decide to do it.

@Zircoz Zircoz closed this Oct 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.