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-119605: Respect follow_wrapped for __init__ and __new__ when getting class signature with inspect.signature #132055

Open
wants to merge 30 commits into
base: main
Choose a base branch
from

Conversation

XuehaiPan
Copy link
Contributor

@XuehaiPan XuehaiPan commented Apr 3, 2025

@XuehaiPan XuehaiPan marked this pull request as ready for review April 3, 2025 17:45
@XuehaiPan
Copy link
Contributor Author

cc @JelleZijlstra

@XuehaiPan XuehaiPan requested a review from JelleZijlstra April 4, 2025 15:43
@python-cla-bot
Copy link

python-cla-bot bot commented Apr 6, 2025

All commit authors signed the Contributor License Agreement.

CLA signed

@bedevere-app
Copy link

bedevere-app bot commented Apr 6, 2025

A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated.

Once you have made the requested changes, please leave a comment on this pull request containing the phrase I have made the requested changes; please review again. I will then notify any core developers who have left a review that you're ready for them to take another look at this pull request.

@XuehaiPan XuehaiPan requested a review from JelleZijlstra April 6, 2025 15:34
@XuehaiPan XuehaiPan changed the title gh-119605: Preserve class signature after wrapping with @warnings.deprecated gh-119605: Respect follow_wrapped for __init__ and __new__ when getting class signature with inspect.signature Apr 6, 2025
@XuehaiPan
Copy link
Contributor Author

cc @serhiy-storchaka for code review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants