Skip to content

Create quicksort.py #25

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

Merged
merged 4 commits into from
Oct 2, 2019
Merged

Create quicksort.py #25

merged 4 commits into from
Oct 2, 2019

Conversation

BubbaBeans
Copy link
Contributor

Quick and dirty implementation of a quicksort algorithm

This pull request is:

  • A new Algorithm
  • An update to an existing algorithm.
  • An error fix
  • Other (Describe below*)

This pull request fixes:

Changes:

Quick and dirty implementation of a quicksort algorithm
Copy link
Owner

@abranhe abranhe left a comment

Choose a reason for hiding this comment

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

Thanks so much for your contribution to The All ▲lgorithms Project. Without people like you submitting Pull Requests we couldn't run this project. You rock @BubbaBeans

But it would be great if you can include the test files, therefore you can read the contributing guide https://github.com/abranhe/allalgorithms-python/blob/master/.github/contributing.md

@abranhe abranhe added Tests needed Test are required, so should be added Documentation needed Documentation is required, so should be added labels Oct 1, 2019
BubbaBeans and others added 3 commits October 1, 2019 16:58
Added the required header, which I originally forgot to include!
Copy link
Owner

@abranhe abranhe left a comment

Choose a reason for hiding this comment

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

Thanks for the docs!

@abranhe abranhe merged commit 4305a29 into abranhe:master Oct 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation needed Documentation is required, so should be added Tests needed Test are required, so should be added
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants