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

bpo-39374: Updated sorting documentation with better key function description #18177

Merged
merged 1 commit into from Jan 25, 2020

Conversation

@brandfilt
Copy link
Contributor

brandfilt commented Jan 25, 2020

@the-knights-who-say-ni

This comment has been minimized.

Copy link

the-knights-who-say-ni commented Jan 25, 2020

Hello, and thanks for your contribution!

I'm a bot set up to make sure that the project can legally accept this contribution by verifying everyone involved has signed the PSF contributor agreement (CLA).

Recognized GitHub username

We couldn't find a bugs.python.org (b.p.o) account corresponding to the following GitHub usernames:

@brandfilt

This might be simply due to a missing "GitHub Name" entry in one's b.p.o account settings. This is necessary for legal reasons before we can look at this contribution. Please follow the steps outlined in the CPython devguide to rectify this issue.

You can check yourself to see if the CLA has been received.

Thanks again for the contribution, we look forward to reviewing it!

Copy link
Member

terryjreedy left a comment

I approve either as is, with (...) added twice, or with it added only once. Raymond, do you have a preference? I will merge if you want.

@terryjreedy

This comment has been minimized.

Copy link
Member

terryjreedy commented Jan 25, 2020

@brandfilt Please sign CLA if you have not. It is really easy. I believe we can merge this without, but 'with' is easier, and we would be set for your next PR.

@brandfilt

This comment has been minimized.

Copy link
Contributor Author

brandfilt commented Jan 25, 2020

@terryjreedy I added my GitHub name to bugs.python.org details and signed the CLA after creating the pull request

@terryjreedy

This comment has been minimized.

Copy link
Member

terryjreedy commented Jan 25, 2020

Thanks. It takes a working day for it to get recorded properly.

@rhettinger

This comment has been minimized.

Copy link
Contributor

rhettinger commented Jan 25, 2020

The Azure Pipeline seems to be stuck. Closing and reopening this PR in an attempt to restart it.

@rhettinger rhettinger closed this Jan 25, 2020
@rhettinger rhettinger reopened this Jan 25, 2020
@rhettinger rhettinger merged commit 8271441 into python:master Jan 25, 2020
6 checks passed
6 checks passed
Docs
Details
Docs
Details
Azure Pipelines PR #20200125.27 succeeded
Details
bedevere/issue-number Issue number 39374 found
Details
bedevere/news "skip news" label found
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
6 participants
You can’t perform that action at this time.