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-125887: Update PyObject_HasAttr exception behavior #125907

Merged
merged 2 commits into from
Dec 27, 2024

Conversation

Damien-Chen
Copy link
Contributor

@Damien-Chen Damien-Chen commented Oct 24, 2024

Copy link
Contributor

@hauntsaninja hauntsaninja left a comment

Choose a reason for hiding this comment

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

I think we should continue to mention the alternatives

Doc/c-api/object.rst Outdated Show resolved Hide resolved
@ZeroIntensity ZeroIntensity added the needs backport to 3.13 bugs and security fixes label Oct 24, 2024
@skirpichev
Copy link
Member

@Damien-Chen, please avoid force-pushes.

@Damien-Chen
Copy link
Contributor Author

I want to rebase and squash my commit, but --force push seems necessary.

@skirpichev
Copy link
Member

Please avoid also rebase, etc. Just add new commits - all going to be squashed in a single commit on merge anyway.

FYI: https://devguide.python.org/getting-started/pull-request-lifecycle/#lifecycle-of-a-pull-request

@hauntsaninja hauntsaninja merged commit 08a0728 into python:main Dec 27, 2024
25 checks passed
@miss-islington-app
Copy link

Thanks @Damien-Chen for the PR, and @hauntsaninja for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 27, 2024
…-125907)

Update PyObject_HasAttr exception behavior

(cherry picked from commit 08a0728)

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

bedevere-app bot commented Dec 27, 2024

GH-128283 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 Dec 27, 2024
hauntsaninja added a commit that referenced this pull request Dec 27, 2024
…) (#128283)

gh-125887: Update PyObject_HasAttr exception behavior (GH-125907)

Update PyObject_HasAttr exception behavior

(cherry picked from commit 08a0728)

Co-authored-by: Damien <[email protected]>
Co-authored-by: Shantanu <[email protected]>
@Damien-Chen Damien-Chen deleted the patch-1 branch January 4, 2025 14:18
srinivasreddy pushed a commit to srinivasreddy/cpython that referenced this pull request Jan 8, 2025
…25907)

Update PyObject_HasAttr exception behavior

Co-authored-by: Shantanu <[email protected]>
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
Status: Done
Development

Successfully merging this pull request may close these issues.

Outdated PyObject_HasAttr documentation (Exception behavior)
4 participants