Skip to content

Double sort (Added with required updates) #1399

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 5 commits into from
Oct 19, 2019

Conversation

ayush246
Copy link
Contributor

I have made all the changes requested .

@ayush246 ayush246 changed the title Added with required updates Double sort (Added with required updates) Oct 19, 2019
@cclauss
Copy link
Member

cclauss commented Oct 19, 2019

Travis CI tests are failing. See CONTRIBUTING.md for details.

@cclauss cclauss added the tests are failing Do not merge until tests pass label Oct 19, 2019
lst[no_of_elements-1-j]=lst[no_of_elements-2-j]
lst[no_of_elements-2-j]=temp
return lst
print("enter the list to be sorted")
Copy link
Member

Choose a reason for hiding this comment

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

Please put lines 29-31 under a if __name__ == "__main__": block to placate Travis CI

lst = [int(x) for x in input().split()] # inputing elements of the list in one line
if __name__ == "__main__":
print("enter the list to be sorted")
lst = [int(x) for x in input().split()] # inputing elements of the list in one line
sorted_lst=double_sort(lst)
Copy link
Member

Choose a reason for hiding this comment

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

Indent the remaining line also or Travis tests will fail.

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.

Awesome!!

@cclauss cclauss merged commit ab65a39 into TheAlgorithms:master Oct 19, 2019
stokhos pushed a commit to stokhos/Python that referenced this pull request Jan 3, 2021
* Added with required updates

* Updated

* required updates

* Update double_sort.py

* Update double_sort.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tests are failing Do not merge until tests pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants