X Tutup
Skip to content

gh-112328: [Enum] Make some private attributes public.#112514

Merged
ethanfurman merged 6 commits intopython:mainfrom
ethanfurman:enum-private_public
Dec 5, 2023
Merged

gh-112328: [Enum] Make some private attributes public.#112514
ethanfurman merged 6 commits intopython:mainfrom
ethanfurman:enum-private_public

Conversation

@ethanfurman
Copy link
Member

@ethanfurman ethanfurman commented Nov 29, 2023 •
edited
Loading

  • _EnumDict --> EnumDict
  • EnumDict._member_names --> EnumDict.member_names
  • Enum._add_alias_
  • Enum._add_value_alias_

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

@ethanfurman
[Enum] Make some private attributes public.
e732a33 
- ``_EnumDict`` --> ``EnumDict``
- ``EnumDict._member_names`` --> ``EnumDict.member_names``
- ``EnumType._add_alias_``
- ``EnumType._add_value_alias_``
@ethanfurman ethanfurman added type-feature A feature request or enhancement tests Tests in the Lib/test dir docs Documentation in the Doc dir stdlib Standard Library Python modules in the Lib/ directory 3.13 bugs and security fixes labels Nov 29, 2023
@ethanfurman ethanfurman requested a review from sobolevn November 29, 2023 05:09
@ethanfurman ethanfurman self-assigned this Nov 29, 2023
@bedevere-app bedevere-app bot mentioned this pull request Nov 29, 2023
Closed
sobolevn
sobolevn reviewed Nov 29, 2023
View reviewed changes
Copy link
Member

@sobolevn sobolevn left a comment

Choose a reason for hiding this comment

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

Thanks for your work! The main question I have, since I am not as deep in the context of enum as you are, what is the main motivation for this? The issue does not have any details :(

Doc/howto/enum.rst Outdated
- :attr:`~Enum._ignore_` -- a list of names, either as a :class:`list` or a
:class:`str`, that will not be transformed into members, and will be removed
from the final class
- :attr:`~Enum._order_` -- used in Python 2/3 code to ensure member order is
Copy link
Member

@sobolevn sobolevn Nov 29, 2023

Choose a reason for hiding this comment

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

I think we can drop Python2 part.

Copy link
Member Author

@ethanfurman ethanfurman Nov 29, 2023

Choose a reason for hiding this comment

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

Keeping for now -- I'll probably change _order_ to be definitive so Enum still works on python implementations without ordered dictionaries.

AlexWaygood
AlexWaygood previously requested changes Nov 29, 2023
View reviewed changes
Copy link
Member

@AlexWaygood AlexWaygood left a comment

Choose a reason for hiding this comment

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

Some of the syntax in enum.rst is incorrect currently:

@bedevere-app
Copy link

bedevere-app bot commented Nov 29, 2023

When you're done making the requested changes, leave the comment: I have made the requested changes; please review again.

ethanfurman
ethanfurman commented Nov 29, 2023
View reviewed changes
ethanfurman
ethanfurman commented Nov 29, 2023
View reviewed changes
@ethanfurman @AlexWaygood
Apply suggestions from code review
b771360 
Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com>
@AlexWaygood AlexWaygood dismissed their stale review November 29, 2023 15:42

Requested changes were made

@ethanfurman
Copy link
Member Author

ethanfurman commented Nov 29, 2023

I have made the requested changes; please review again.

I also moved the two _add_* methods from EnumType to Enum, and fixed unhashable value handling.

@ethanfurman ethanfurman merged commit de6bca9 into python:main Dec 5, 2023
@ethanfurman ethanfurman deleted the enum-private_public branch December 5, 2023 16:27
aisk pushed a commit to aisk/cpython that referenced this pull request Feb 11, 2024
@ethanfurman @AlexWaygood
@sobolevn @aisk
pythongh-112328: [Enum] Make some private attributes public. (pythonG…
23f9995 
…H-112514)

* [Enum] Make some private attributes public.

- ``_EnumDict`` --> ``EnumDict``
- ``EnumDict._member_names`` --> ``EnumDict.member_names``
- ``Enum._add_alias_``
- ``Enum._add_value_alias_``

---------

Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com>
Co-authored-by: Nikita Sobolev <mail@sobolevn.me>
@ethanfurman ethanfurman added 3.12 only security fixes needs backport to 3.12 only security fixes labels Apr 9, 2024
@miss-islington-app
Copy link

miss-islington-app bot commented Apr 9, 2024

Thanks @ethanfurman for the PR 🌮🎉.. I'm working now to backport this PR to: 3.12.
🐍🍒⛏🤖

@miss-islington-app
Copy link

miss-islington-app bot commented Apr 9, 2024

Sorry, @ethanfurman, I could not cleanly backport this to 3.12 due to a conflict.
Please backport using cherry_picker on command line.

cherry_picker de6bca956432cc852a4a41e2a2cee9cdacd19f35 3.12

@ethanfurman
Copy link
Member Author

ethanfurman commented Apr 9, 2024

Backporting because changes are needed for a bug-fix.

@aminalaee aminalaee mentioned this pull request May 27, 2024
Merged
@u2rafi u2rafi mentioned this pull request Jul 13, 2024
Merged
Glyphack pushed a commit to Glyphack/cpython that referenced this pull request Sep 2, 2024
@ethanfurman @AlexWaygood
@sobolevn @Glyphack
pythongh-112328: [Enum] Make some private attributes public. (pythonG…
54c174f 
…H-112514)

* [Enum] Make some private attributes public.

- ``_EnumDict`` --> ``EnumDict``
- ``EnumDict._member_names`` --> ``EnumDict.member_names``
- ``Enum._add_alias_``
- ``Enum._add_value_alias_``

---------

Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com>
Co-authored-by: Nikita Sobolev <mail@sobolevn.me>
@alien-valkyrie alien-valkyrie mentioned this pull request Oct 18, 2024
Closed
@hugovk
Copy link
Member

hugovk commented Feb 26, 2025

Backporting because changes are needed for a bug-fix.

@ethanfurman Triage: Is this still the case? If so, please could you create the 3.12 backport? Otherwise let's remove the backport label.

@hugovk
Copy link
Member

hugovk commented Apr 8, 2025

3.12 is now security-only, so the backport is no longer needed.

https://discuss.python.org/t/python-3-12-is-now-security-fix-only/87520

@hugovk hugovk removed 3.12 only security fixes needs backport to 3.12 only security fixes labels Apr 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sobolevn sobolevn sobolevn left review comments

@AlexWaygood AlexWaygood AlexWaygood left review comments

Assignees

@ethanfurman ethanfurman

Labels

3.13 bugs and security fixes docs Documentation in the Doc dir stdlib Standard Library Python modules in the Lib/ directory tests Tests in the Lib/test dir type-feature A feature request or enhancement

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@ethanfurman @hugovk @sobolevn @AlexWaygood
X Tutup