Hello, and thanks for your contribution!

I'm a bot set up to make sure that the project can legally accept your contribution by verifying you have signed thePSF contributor agreement (CLA).

Unfortunately our records indicate you have not signed the CLA. For legal reasons we need you to sign this before we can look at your contribution. Please followthe steps outlined in the CPython devguide to rectify this issue.

When your account is ready, please add a comment in this pull request
and a Python core developer will remove theCLA not signed label
to make the bot check again.

Thanks again for your contribution, we look forward to reviewing it!

CLA is signed and pending.

I don't think this is backwards-compatible- users might create an enum to refer to types deliberately:

classMyTypes(Enum):i=intf=floats=str

>>>MyTypes.i<MyTypes.i:<class'int'>>>>>MyTypes.i.value<class'int'>

This is perhaps a little strange, but I don't think your use case is any less strange, and status quo wins.

Perhaps the check should be for classes actually declared within the enum, using__qual_name__

@eric-wieser Thanks for the catch. Added a unit test for your example and updated the PR to fix it.

@ethanfurman Were you able to find a moment to look further into this PR? Thanks again in advance.

I have been thinking about it. At this point I'm still not sure how I want to resolve the issue. I'm thinking about having embedded/referenced classes be part of anEnum value, but also be directly callable.

For example:

MyTypes.i()

would result in0.

For those times when one does not want a class as a member, it's easy enough to write an@skip decorater -- in fact, one is included inaenum.

Edward, thanks for your work. I decided to go a different route where the work is done in_EnumDict instead (which you can find in myaenum project).

I'm closing this PR, but if you are up for a challenge I need a new PR that:

• adds an@member decorator to force an object to become a member
• adds an@nonmember decorator to protect an object from becoming a member
• adds the_is_class() utility for deciding if an object is a class
• updatesEnumMeta to forward the class name to_EnumDict, and_EnumDict to store and forward the class name to_is_class()
• issues aDeprecationWarning when a bare class is present in anEnum, and still convert it to a member (warning should mention coming change in behavior, and the@member decorator to maintain the current behavior)

Edward, thanks for your work. I decided to go a different route where the work is done in_EnumDict instead (which you can find in myaenum project).

I'm closing this PR, but if you are up for a challenge I need a new PR that:

• adds an@member decorator to force an object to become a member

• adds an@nonmember decorator to protect an object from becoming a member

• adds the_is_class() utility for deciding if an object is a class

• updatesEnumMeta to forward the class name to_EnumDict, and_EnumDict to store and forward the class name to_is_class()

• issues aDeprecationWarning when a bare class is present in anEnum, and still convert it to a member (warning should mention coming change in behavior, and the@member decorator to maintain the current behavior)

@ethanfurman
Are these changes still required?

@edwardcwang Hey, I'm adding you to the ACKS file as "Edward C Wang" -- let me know if that should be something different.

"Edward Wang" would actually be fine too!