Skip to content
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

gh-131045: [Enum] fix flag containment checks when using values #131053

Merged
merged 4 commits into from
Mar 12, 2025

Conversation

ethanfurman
Copy link
Member

@ethanfurman ethanfurman commented Mar 10, 2025

Check would fail if value would create a pseudo-member, but that member had not yet been created. We now attempt to create a pseudo-member for a passed-in value first.

Check would fail if value would create a pseudo-member, but that member
had not yet been created.  We now attempt to create a pseudo-member for
a passed-in value first.
@ethanfurman ethanfurman added type-bug An unexpected behavior, bug, or error stdlib Python modules in the Lib dir 3.12 bugs and security fixes 3.13 bugs and security fixes needs backport to 3.12 bug and security fixes 3.14 new features, bugs and security fixes needs backport to 3.13 bugs and security fixes labels Mar 10, 2025
@ethanfurman ethanfurman self-assigned this Mar 10, 2025
ethanfurman and others added 3 commits March 11, 2025 13:26
spelling and mark-up fixes

Co-authored-by: Tomas R. <tomas.roun8@gmail.com>
Co-authored-by: Tomas R. <tomas.roun8@gmail.com>
@picnixz picnixz removed 3.12 bugs and security fixes 3.13 bugs and security fixes 3.14 new features, bugs and security fixes type-bug An unexpected behavior, bug, or error stdlib Python modules in the Lib dir labels Mar 12, 2025
@picnixz
Copy link
Member

picnixz commented Mar 12, 2025

(We generally reserve those labels to issues)

@ethanfurman ethanfurman merged commit 17d06ae into python:main Mar 12, 2025
49 checks passed
@miss-islington-app
Copy link

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

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Mar 12, 2025
…pythonGH-131053)

Check would fail if value would create a pseudo-member, but that member
had not yet been created.  We now attempt to create a pseudo-member for
a passed-in value first.
(cherry picked from commit 17d06ae)

Co-authored-by: Ethan Furman <ethan@stoneleaf.us>
Co-authored-by: Tomas R. <tomas.roun8@gmail.com>
@miss-islington-app
Copy link

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 17d06aeb5476099bc1acd89cd6f69e239e0f9350 3.12

@bedevere-app
Copy link

bedevere-app bot commented Mar 12, 2025

GH-131167 is a backport of this pull request to the 3.13 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.13 bugs and security fixes label Mar 12, 2025
@picnixz
Copy link
Member

picnixz commented Mar 13, 2025

The 3.12 bp failed, do you want someone else to make it or are you planning to do it? @ethanfurman

@bedevere-app
Copy link

bedevere-app bot commented Mar 14, 2025

GH-131232 is a backport of this pull request to the 3.12 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.12 bug and security fixes label Mar 14, 2025
plashchynski pushed a commit to plashchynski/cpython that referenced this pull request Mar 17, 2025
…pythonGH-131053)

Check would fail if value would create a pseudo-member, but that member
had not yet been created.  We now attempt to create a pseudo-member for
a passed-in value first.

Co-authored-by: Tomas R. <tomas.roun8@gmail.com>
picnixz pushed a commit that referenced this pull request Mar 23, 2025
GH-131053) (#131167)

gh-131045: [Enum] fix flag containment checks when using values (GH-131053)

Check would fail if value would create a pseudo-member, but that member
had not yet been created.  We now attempt to create a pseudo-member for
a passed-in value first.
(cherry picked from commit 17d06ae)

Co-authored-by: Ethan Furman <ethan@stoneleaf.us>
Co-authored-by: Tomas R. <tomas.roun8@gmail.com>
picnixz added a commit that referenced this pull request Mar 24, 2025
GH-131053) (#131232)

* gh-131045: [Enum] fix flag containment checks when using values (GH-131053)

Check would fail if value would create a pseudo-member, but that member
had not yet been created.  We now attempt to create a pseudo-member for
a passed-in value first.

---------

Co-authored-by: Ethan Furman <ethan@stoneleaf.us>
Co-authored-by: Tomas R. <tomas.roun8@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants