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

Added Selection Sort in C++ #364

Merged
merged 4 commits into from
Oct 27, 2018
Merged

Conversation

susantruong
Copy link
Contributor

Description

Added two files named '.vscode/tasks.json" and "SelectionSort" into this path "Sorting/Selection Sort/C++/" that implement the Selection Sort function in C++ .

Related Issue

Link to the issue here: #363

Types of changes

What types of changes does your code introduce? Put an x in all the boxes that apply:

  • Bug fix (change which fixes an issue with the algorithm)

  • New Algorithm (non-breaking change which adds functionality)

  • Documentation

Checklist:

Go over all the following points, and put an x in all the boxes that apply.

If you're unsure about any of these, don't hesitate to ask. We're here to help!

  • My code follows the code style of this project.

  • My change requires a change to the documentation.

  • I have updated the documentation accordingly.

  • I have read the CONTRIBUTING document.

@susantruong
Copy link
Contributor Author

@Asiatik This issue needs two reviewers. Would you please review it and ask your team members to review it? Many thanks for your help. :)

@@ -0,0 +1,42 @@
#include <iostream>
#include <cstdlib>
Copy link
Member

Choose a reason for hiding this comment

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

I guess you don't need to include cstdlib.

@tstreamDOTh
Copy link
Member

Hi @susantruong. You can join Asiatik using https://orgmanager.miguelpiedrafita.com/join/36604391

@tstreamDOTh tstreamDOTh merged commit 087ce79 into Asiatik:master Oct 27, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants