Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Fixed _get_slots bug, according to issue gh-105866 #105870

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

Merged
merged 10 commits into from
Mar 19, 2024

Conversation

avboag
Copy link
Contributor

@avboag avboag commented Jun 16, 2023

Fixed _get_slots bug, according to issue gh-105866

#105866: Fixed _get_slots to account for the implicitly defined slots.

@bedevere-bot
Copy link

Most changes to Python require a NEWS entry.

Please add it using the blurb_it web app or the blurb command-line tool.

@ghost
Copy link

ghost commented Jun 16, 2023

All commit authors signed the Contributor License Agreement.
CLA signed

@avboag avboag changed the title Fixed _get_slots bug. Fixed _get_slots bug, according to issue #105866 Jun 16, 2023
@avboag avboag changed the title Fixed _get_slots bug, according to issue #105866 Fixed _get_slots bug, according to issue gh-105866 Jun 16, 2023
@Eclips4
Copy link
Member

Eclips4 commented Jun 16, 2023

Also, it would be nice to have tests for this case

Copy link
Member

@carljm carljm left a comment

Choose a reason for hiding this comment

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

This looks good to me.

@carljm carljm merged commit a22d05f into python:main Mar 19, 2024
@carljm carljm added needs backport to 3.11 only security fixes needs backport to 3.12 only security fixes labels Mar 19, 2024
@miss-islington-app
Copy link

Thanks @avboag for the PR, and @carljm for merging it 🌮🎉.. I'm working now to backport this PR to: 3.12.
🐍🍒⛏🤖

@miss-islington-app
Copy link

Thanks @avboag for the PR, and @carljm for merging it 🌮🎉.. I'm working now to backport this PR to: 3.11.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Mar 19, 2024
…thonGH-105870)

(cherry picked from commit a22d05f)

Co-authored-by: Aviel Boag <[email protected]>
Co-authored-by: blurb-it[bot] <43283697+blurb-it[bot]@users.noreply.github.com>
Co-authored-by: Kirill Podoprigora <[email protected]>
Co-authored-by: Carl Meyer <[email protected]>
@bedevere-app
Copy link

bedevere-app bot commented Mar 19, 2024

GH-116978 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 19, 2024
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Mar 19, 2024
…thonGH-105870)

(cherry picked from commit a22d05f)

Co-authored-by: Aviel Boag <[email protected]>
Co-authored-by: blurb-it[bot] <43283697+blurb-it[bot]@users.noreply.github.com>
Co-authored-by: Kirill Podoprigora <[email protected]>
Co-authored-by: Carl Meyer <[email protected]>
@bedevere-app
Copy link

bedevere-app bot commented Mar 19, 2024

GH-116979 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 19, 2024
ambv pushed a commit that referenced this pull request Mar 19, 2024
…H-105870) (GH-116978)

(cherry picked from commit a22d05f)

Co-authored-by: Aviel Boag <[email protected]>
Co-authored-by: Kirill Podoprigora <[email protected]>
Co-authored-by: Carl Meyer <[email protected]>
ambv pushed a commit that referenced this pull request Mar 19, 2024
…H-105870) (GH-116979)

(cherry picked from commit a22d05f)

Co-authored-by: Aviel Boag <[email protected]>
Co-authored-by: Kirill Podoprigora <[email protected]>
Co-authored-by: Carl Meyer <[email protected]>
vstinner pushed a commit to vstinner/cpython that referenced this pull request Mar 20, 2024
…thon#105870)

Co-authored-by: blurb-it[bot] <43283697+blurb-it[bot]@users.noreply.github.com>
Co-authored-by: Kirill Podoprigora <[email protected]>
Co-authored-by: Carl Meyer <[email protected]>
adorilson pushed a commit to adorilson/cpython that referenced this pull request Mar 25, 2024
…thon#105870)

Co-authored-by: blurb-it[bot] <43283697+blurb-it[bot]@users.noreply.github.com>
Co-authored-by: Kirill Podoprigora <[email protected]>
Co-authored-by: Carl Meyer <[email protected]>
diegorusso pushed a commit to diegorusso/cpython that referenced this pull request Apr 17, 2024
…thon#105870)

Co-authored-by: blurb-it[bot] <43283697+blurb-it[bot]@users.noreply.github.com>
Co-authored-by: Kirill Podoprigora <[email protected]>
Co-authored-by: Carl Meyer <[email protected]>
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.

4 participants