Skip to content

Fix #82: Implement the 'reverse' option for sorted() #119

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

Closed
wants to merge 1 commit into from

Conversation

alanjds
Copy link

@alanjds alanjds commented Oct 29, 2018

This is the companion PR of #82: Implement the 'reverse' option for sorted()

See comments on #82, if any

Original:

google#162 opened on Jan 18, 2017 by @MirkoDziadzka

Add the 'reverse' option to 'sorted()'

I'm not sure if this violates the 'stable sort definition in Python.
On the other hand, this is probably only relevant when key or cmp gets implemented.

@alanjds alanjds requested a review from a team October 29, 2018 02:11
@funny-falcon
Copy link

Sorry, I could not see any difference from already merged #102 , and I saw its code merged into master ... but this branch has no conflicts... What's happening? Maybe it is because I'm from phone, and web-mobile github version doesn't show me some important details?

@alanjds
Copy link
Author

alanjds commented Oct 29, 2018

You are right. I was cleaning the open branches and found this one.

Forgot that the feature was already merged. Sorry and thanks for pointing.

@alanjds alanjds closed this Oct 29, 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.

2 participants