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.

Sign up for GitHub

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

Jump to bottom

Added Merge Sort [C#] #492

Merged
merged 3 commits into from
Oct 5, 2017
Merged

Added Merge Sort [C#] #492

merged 3 commits into from
Oct 5, 2017

Conversation

DimaMukhin
Copy link
Contributor

Fixes #473

By submitting this pull request I confirm I've read and complied with the below declarations.

  • I have read the Contribution guidelines and I am confident that my PR reflects them.
  • I have followed the coding guidelines for this project.
  • My code follows the skeleton code structure.
  • This pull request has a descriptive title. For example, Added {Algorithm/DS name} [{Language}], not Update README.md or Added new code.
  • This pull request will be closed if I fail to update it even once in a continuous time span of 7 days.

@singhpratyush
Copy link
Member

Hi! Thanks for showing interest in contributing to this repository.

Please make sure that you have ticked the points mentioned in PR description correctly.

Thanks again!

@DimaMukhin
Copy link
Contributor Author

Looks like gitmate broke down but I cannot find out why. Can this be ignored? Where can I find details about it or restart the test?

mohitkyadav
mohitkyadav requested changes Oct 4, 2017
View reviewed changes
Copy link
Member

@mohitkyadav mohitkyadav left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mention the time and space complexity in MergeSort.cs file. Also add function comments.

@DimaMukhin DimaMukhin mentioned this pull request Oct 4, 2017
Closed
5 tasks
@singhpratyush singhpratyush merged commit d7d0256 into iiitv:master Oct 5, 2017
@DimaMukhin DimaMukhin deleted the merge-sort-c# branch October 7, 2017 18:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mohitkyadav mohitkyadav mohitkyadav approved these changes

Assignees
No one assigned
Labels
Algorithm difficulty-easy lang-csharp need review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Merge Sort [C#]
3 participants
@DimaMukhin @singhpratyush @mohitkyadav