Skip to content

gh-105535 Document potential performance trap during enum creation#107119

Merged
ethanfurman merged 11 commits into
python:mainfrom
nicky-eng:fix-issue-105535
Mar 8, 2024
Merged

gh-105535 Document potential performance trap during enum creation#107119
ethanfurman merged 11 commits into
python:mainfrom
nicky-eng:fix-issue-105535

Conversation

@nicky-eng

@nicky-eng nicky-eng commented Jul 23, 2023

Copy link
Copy Markdown
Contributor

@ghost

ghost commented Jul 23, 2023

Copy link
Copy Markdown

All commit authors signed the Contributor License Agreement.
CLA signed

Comment thread Doc/library/enum.rst Outdated
Comment thread Doc/library/enum.rst Outdated
@ethanfurman ethanfurman requested a review from CAM-Gerlach July 24, 2023 18:45
Comment thread Doc/library/enum.rst Outdated
@ethanfurman ethanfurman self-assigned this Jul 24, 2023
Comment thread Doc/library/enum.rst Outdated
@ethanfurman

Copy link
Copy Markdown
Member

Thoughts on making this an actual warning box?

Comment thread Doc/library/enum.rst Outdated
Comment thread Doc/library/enum.rst Outdated
attempt to fix dataclass link
Comment thread Doc/library/enum.rst Outdated
Comment thread Doc/library/enum.rst Outdated
Comment thread Doc/library/enum.rst
Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
@ethanfurman

Copy link
Copy Markdown
Member

@hugovk, thanks, I missed that those were the sunder versions when I merged them.

Comment thread Doc/library/enum.rst Outdated
Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
@ethanfurman ethanfurman merged commit 601f3a7 into python:main Mar 8, 2024
@miss-islington-app

Copy link
Copy Markdown

Thanks @nicky-eng for the PR, and @ethanfurman for merging it 🌮🎉.. I'm working now to backport this PR to: 3.11, 3.12.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Mar 8, 2024
…ion (pythonGH-107119)

(cherry picked from commit 601f3a7)

Co-authored-by: Nicolas A. Oyarzabal <79150521+nicky-eng@users.noreply.github.com>
Co-authored-by: Ethan Furman <ethan@stoneleaf.us>
Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
@bedevere-app

bedevere-app Bot commented Mar 8, 2024

Copy link
Copy Markdown

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

@bedevere-app bedevere-app Bot removed the needs backport to 3.12 only security fixes label Mar 8, 2024
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Mar 8, 2024
…ion (pythonGH-107119)

(cherry picked from commit 601f3a7)

Co-authored-by: Nicolas A. Oyarzabal <79150521+nicky-eng@users.noreply.github.com>
Co-authored-by: Ethan Furman <ethan@stoneleaf.us>
Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
@bedevere-app

bedevere-app Bot commented Mar 8, 2024

Copy link
Copy Markdown

GH-116512 is a backport of this pull request to the 3.11 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.11 only security fixes label Mar 8, 2024
ethanfurman added a commit that referenced this pull request Mar 9, 2024
…tion (GH-107119) (GH116511)

(cherry picked from commit 601f3a7)

Co-authored-by: Nicolas A. Oyarzabal <79150521+nicky-eng@users.noreply.github.com>
Co-authored-by: Ethan Furman <ethan@stoneleaf.us>
Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
ethanfurman added a commit that referenced this pull request Mar 9, 2024
…tion (GH-107119) (GH-116512)

(cherry picked from commit 601f3a7)

Co-authored-by: Nicolas A. Oyarzabal <79150521+nicky-eng@users.noreply.github.com>
Co-authored-by: Ethan Furman <ethan@stoneleaf.us>
Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
adorilson pushed a commit to adorilson/cpython that referenced this pull request Mar 25, 2024
…ion (pythonGH-107119)

Co-authored-by: Ethan Furman <ethan@stoneleaf.us>
Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
diegorusso pushed a commit to diegorusso/cpython that referenced this pull request Apr 17, 2024
…ion (pythonGH-107119)

Co-authored-by: Ethan Furman <ethan@stoneleaf.us>
Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation in the Doc dir skip news

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants