Skip to content

Conversation

serhiy-storchaka
Copy link
Member

@serhiy-storchaka serhiy-storchaka commented Feb 14, 2025

Copy link
Contributor

@skirpichev skirpichev left a comment

Choose a reason for hiding this comment

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

If we are going for changes in slot_nb_inplace_power(), I think this must be tested.

The rest looks fine.

@serhiy-storchaka serhiy-storchaka enabled auto-merge (squash) February 16, 2025 10:01
@serhiy-storchaka serhiy-storchaka merged commit cfe4103 into python:main Feb 17, 2025
40 checks passed
@miss-islington-app
Copy link

Thanks @serhiy-storchaka 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 Feb 17, 2025
Test it with the third argument.
(cherry picked from commit cfe4103)

Co-authored-by: Serhiy Storchaka <[email protected]>
@bedevere-app
Copy link

bedevere-app bot commented Feb 17, 2025

GH-130211 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 Feb 17, 2025
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Feb 17, 2025
Test it with the third argument.
(cherry picked from commit cfe4103)

Co-authored-by: Serhiy Storchaka <[email protected]>
@bedevere-app
Copy link

bedevere-app bot commented Feb 17, 2025

GH-130212 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 Feb 17, 2025
serhiy-storchaka added a commit that referenced this pull request Feb 17, 2025
…H-130211)

Test it with the third argument.
(cherry picked from commit cfe4103)

Co-authored-by: Serhiy Storchaka <[email protected]>
serhiy-storchaka added a commit that referenced this pull request Feb 17, 2025
…H-130212)

Test it with the third argument.
(cherry picked from commit cfe4103)

Co-authored-by: Serhiy Storchaka <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip news tests Tests in the Lib/test dir
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants