3.13 `copy.replace` isn't in `__all__`

[max-muoto]

Bug report

Bug description:

copy.replace isn't exported as a part of __all__ in the copy module. This seemed liked an oversight, as opposed to being intentional, but feel free to close this issue if not.

CPython versions tested on:

3.13

Operating systems tested on:

No response

Linked PRs

• gh-121300: Add replace to copy.__all__ #121302
• gh-121300: Update __all__ to include copy.replace #121304
• [3.13] gh-121300: Add replace to copy.__all__ (GH-121302) #121337

[max-muoto]

CC: @serhiy-storchaka

[pygeek]

Nice catch:

#108752

[Eclips4]

Hello!
Nice catch. This definitely should be in __all__ since this is part of the public API.
Would you like to send a PR with fix?

[max-muoto]

Hello!

Nice catch. This definitely should be in __all__ since this is part of the public API.

Would you like to send a PR with fix?

Will do! Thanks for catching @JelleZijlstra!

[max-muoto]

@Eclips4 #121302

[serhiy-storchaka]

Sure, this is an oversight. pydoc copy should show replace.

[max-muoto]

Sure, this is an oversight. pydoc copy should show replace.

Do you want a separate PR here (adding it to the module docstring)? Or can add it into the existing one I have.

[pygeek]

@Eclips4 #121302

I know this is a trivial change, but we should probably add a regression test for __all__. See support.test__all__ helper method in the test directory.

[serhiy-storchaka]

Adding it to the module docstring is also a good idea. You can include this in your PR or create a separate PR if you wish.

[max-muoto]

Adding it to the module docstring is also a good idea. You can include this in your PR or create a separate PR if you wish.

Let me just do that in the current PR, I'm also going to add a test case.