Skip to content

adding input option and increasing the number of doctest #1281

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 2 commits into from
Oct 6, 2019
Merged

adding input option and increasing the number of doctest #1281

merged 2 commits into from
Oct 6, 2019

Conversation

mvhb
Copy link
Contributor

@mvhb mvhb commented Oct 6, 2019

No description provided.

@mvhb mvhb changed the title adding input option and incresing the number of doctest adding input option and increasing the number of doctest Oct 6, 2019
Copy link
Member

@cclauss cclauss left a comment

Choose a reason for hiding this comment

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

One test change please

>>> stooge_sort([])
[]

>>> stooge_sort([-2, -5, -45])
Copy link
Member

Choose a reason for hiding this comment

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

Good but let’s mix positive int, positive float, zero, negative int, and negative float all in one test.

@cclauss cclauss closed this Oct 6, 2019
@mvhb
Copy link
Contributor Author

mvhb commented Oct 6, 2019

hi @cclauss, will this code be merged?
thank you!

@cclauss cclauss reopened this Oct 6, 2019
@cclauss cclauss merged commit 067a9b5 into TheAlgorithms:master Oct 6, 2019
@cclauss
Copy link
Member

cclauss commented Oct 6, 2019

Done. Thx

stokhos pushed a commit to stokhos/Python that referenced this pull request Jan 3, 2021
…ms#1281)

* adding input option and incresing the number of doctest

* mixing positive and negative numbers in the same test case
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.

2 participants