Skip to content

Replace 'currying' by 'partial function'. #91741

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

clemens-tolboom
Copy link
Contributor

Currying is not mentioned in the text so the reference is a little pointless.

Currying is related to, but not the same as, partial application.

From https://en.wikipedia.org/wiki/Currying.

It is better to replace this with link to https://en.wikipedia.org/wiki/Partial_application as that is used in https://docs.python.org/3.8/howto/functional.html#the-functools-module

Currying is not mentioned in the text so the reference is a little pointless.

> Currying is related to, but not the same as, partial application.

From https://en.wikipedia.org/wiki/Currying.

It is better to replace this with link to https://en.wikipedia.org/wiki/Partial_application as that is used in https://docs.python.org/3.8/howto/functional.html#the-functools-module
@ghost
Copy link

ghost commented Apr 20, 2022

All commit authors signed the Contributor License Agreement.
CLA signed

@bedevere-bot
Copy link

Every change to Python requires a NEWS entry.

Please, add it using the blurb_it Web app or the blurb command-line tool.

@bedevere-bot bedevere-bot added docs Documentation in the Doc dir awaiting review labels Apr 20, 2022
@AlexWaygood
Copy link
Member

Hi, thanks for the PR! It looks like you're trying to have this PR merged into the 3.8 branch. 3.8 is now only accepting security-related patches, so I'm closing this PR.

99% of changes to Python are first made to the main branch, and then backported to 3.10/3.9 (and 3.8/3.7, if the change is security-related). If you want a CPython PR to be merged, I recommend you use the main branch as the main branch 🙂

@clemens-tolboom
Copy link
Contributor Author

Sorry for the wrong branch ... my bad :-/

New main branch version is #91814

@clemens-tolboom clemens-tolboom deleted the patch-1 branch April 22, 2022 08:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting review docs Documentation in the Doc dir
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants