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-39200: Correct the error message for min/max builtin function #17814

Merged
merged 5 commits into from Jan 10, 2020

Conversation

@corona10
Copy link
Member

corona10 commented Jan 3, 2020

@tirkarthi

This comment has been minimized.

Copy link
Contributor

tirkarthi commented Jan 3, 2020

Seems like a duplicate of #17813

@tirkarthi

This comment has been minimized.

Copy link
Contributor

tirkarthi commented Jan 3, 2020

Sorry, misread the diff and PR linked to issue as duplicate solution.

@corona10

This comment has been minimized.

Copy link
Member Author

corona10 commented Jan 3, 2020

@tirkarthi

Sorry, misread the diff and PR linked to issue as duplicate solution.

Yes, IMHO, The issue title should be updated as I wrote at the bpo-39200

@pablogsal

This comment has been minimized.

Copy link
Member

pablogsal commented Jan 3, 2020

Let's hold until we decide how to go in #17813

corona10 added 3 commits Jan 4, 2020
Python/bltinmodule.c Outdated Show resolved Hide resolved
Copy link
Member

pablogsal left a comment

LGTM

@corona10

This comment has been minimized.

Copy link
Member Author

corona10 commented Jan 7, 2020

@pablogsal Updated :)

@corona10 corona10 requested a review from pablogsal Jan 7, 2020
@corona10 corona10 requested a review from vstinner Jan 10, 2020
@vstinner vstinner merged commit abdc634 into python:master Jan 10, 2020
9 checks passed
9 checks passed
Docs
Details
Windows (x86)
Details
Windows (x64)
Details
macOS
Details
Ubuntu
Details
Azure Pipelines PR #20200107.4 succeeded
Details
bedevere/issue-number Issue number 39200 found
Details
bedevere/news News entry found in Misc/NEWS.d
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@miss-islington

This comment has been minimized.

Copy link

miss-islington commented Jan 10, 2020

Thanks @corona10 for the PR, and @vstinner for merging it 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒🤖

miss-islington added a commit to miss-islington/cpython that referenced this pull request Jan 10, 2020
…thonGH-17814)

Correct the error message when calling the min() or max() with
no arguments.
(cherry picked from commit abdc634)

Co-authored-by: Dong-hee Na <donghee.na92@gmail.com>
@bedevere-bot

This comment has been minimized.

Copy link

bedevere-bot commented Jan 10, 2020

GH-17941 is a backport of this pull request to the 3.8 branch.

@miss-islington

This comment has been minimized.

Copy link

miss-islington commented Jan 10, 2020

Thanks @corona10 for the PR, and @vstinner for merging it 🌮🎉.. I'm working now to backport this PR to: 3.7.
🐍🍒🤖

miss-islington added a commit to miss-islington/cpython that referenced this pull request Jan 10, 2020
…thonGH-17814)

Correct the error message when calling the min() or max() with
no arguments.
(cherry picked from commit abdc634)

Co-authored-by: Dong-hee Na <donghee.na92@gmail.com>
@bedevere-bot

This comment has been minimized.

Copy link

bedevere-bot commented Jan 10, 2020

GH-17943 is a backport of this pull request to the 3.7 branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
7 participants
You can’t perform that action at this time.