-
-
Notifications
You must be signed in to change notification settings - Fork 32k
gh-109961: Use proper module
for copy
method docs
#110027
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
Conversation
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.
Other anchors for object methods do not contain "builtins." prefix. For example https://docs.python.org/3/reference/datamodel.html#object.__format__.
Did you try to use .. currentmodule:: .
? I am not sure whether :module: .
works.
@serhiy-storchaka hm, indeed. Will try that as well! |
The result is identical. Please, take a look. The difference with |
Co-authored-by: Adam Turner <[email protected]>
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.
Thank you!
A
@sobolevn -- when you do the backports, please could you include this commit in them? Saves merge conflicts etc. A |
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.
Thanks!
…10027) Co-authored-by: Adam Turner <[email protected]>
This sets correct
:module:
context forbuiltins.object.{__copy__,__deepcopy__,__replace__}
Now their urls are:
library/copy.html#object.__copy__
library/copy.html#object.__deepcopy__
library/copy.html#object.__replace__
All links seem to work.
CC @AA-Turner @hugovk @serhiy-storchaka
Refs #109968
__replace__
method incopy
docs: consider addingcopy.SupportsReplace
? #109961📚 Documentation preview 📚: https://cpython-previews--110027.org.readthedocs.build/en/110027/library/copy.html