Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork33.3k
gh-141004: DocumentPyFunction_SetKwDefaults#141294
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Conversation
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
I'm sleepy.
| Set the keyword-only argument default values of the function object *op*. | ||
| *defaults* must be a dictionary of keyword-only arguments or ``Py_None``. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
| Set the keyword-onlyargument default values of the function object *op*. | |
| *defaults* must be a dictionary of keyword-onlyarguments or ``Py_None``. | |
| Set the keyword-onlyparameter default values of the function object *op*. | |
| *defaults* must be a dictionary of keyword-onlyparameters or ``Py_None``. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
We say "argument" everywhere else in this file, so I'd rather not deviate from it here. I don't think that the distinction is particularly helpful for readers here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
I'm not a fan of such a precedent, but unfortunately it is. Fixing this is tracked in#133438.
StanFromIreland left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
LGTM
18529b5 intopython:mainUh oh!
There was an error while loading.Please reload this page.
Thanks@ZeroIntensity for the PR 🌮🎉.. I'm working now to backport this PR to: 3.13, 3.14. |
(cherry picked from commit 18529b580b59b8d075641da6c685bef377eb0a7b)Co-authored-by: Peter Bierma <zintensitydev@gmail.com>
GH-141304 is a backport of this pull request to the3.14 branch. |
(cherry picked from commit18529b5)Co-authored-by: Peter Bierma <zintensitydev@gmail.com>
GH-141305 is a backport of this pull request to the3.13 branch. |
Uh oh!
There was an error while loading.Please reload this page.
📚 Documentation preview 📚:https://cpython-previews--141294.org.readthedocs.build/en/141294/c-api/function.html#c.PyFunction_SetKwDefaults