Skip to content

Docs: update docs of copy#105547

Open
shihai1991 wants to merge 2 commits into
python:mainfrom
shihai1991:update_copy_doc
Open

Docs: update docs of copy#105547
shihai1991 wants to merge 2 commits into
python:mainfrom
shihai1991:update_copy_doc

Conversation

@shihai1991
Copy link
Copy Markdown
Member

@shihai1991 shihai1991 commented Jun 9, 2023

Updating reason: I meet many python developers who use copy module, but they don't know the copy module don't copy or deepcopy the immutable object. So I update this doc simply to make sure the user undertand this point clearly.


📚 Documentation preview 📚: https://cpython-previews--105547.org.readthedocs.build/

@bedevere-bot bedevere-bot added awaiting review docs Documentation in the Doc dir skip news labels Jun 9, 2023
@shihai1991 shihai1991 added skip issue and removed docs Documentation in the Doc dir skip news labels Jun 9, 2023
@shihai1991 shihai1991 requested a review from avassalotti June 9, 2023 07:43
@erlend-aasland erlend-aasland changed the title [docs] update docs of copy Docs: update docs of copy Jan 12, 2024
@github-actions
Copy link
Copy Markdown

This PR is stale because it has been open for 30 days with no activity.

@github-actions github-actions Bot added the stale Stale PR or inactive for long period of time. label Apr 13, 2026
Copy link
Copy Markdown
Member

@serhiy-storchaka serhiy-storchaka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is incorrect place for "immutable object". "Does not copy" means that it raises an exception for such types. For immutable objects the behavior is similar to functions and classes, except that it is natural and does not need rational. So, if add clarification for immutable objects, I would insert a separate sentence between the cases "does not copy" and "copy" in quotes. The latter would need to be modified to avoid repeating.

@bedevere-app
Copy link
Copy Markdown

bedevere-app Bot commented May 23, 2026

A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated.

Once you have made the requested changes, please leave a comment on this pull request containing the phrase I have made the requested changes; please review again. I will then notify any core developers who have left a review that you're ready for them to take another look at this pull request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting changes skip issue skip news stale Stale PR or inactive for long period of time.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants