-
-
Notifications
You must be signed in to change notification settings - Fork 2.8k
Switch to new-style pluggy hook wrappers #11123
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's wait on pytest-dev/pluggy#405 before moving this forward. 👍
52c1a7e
to
147ad6b
Compare
Updated, pending pluggy 1.2.0 release (the pluggymain variants should pass). |
147ad6b
to
d9290ce
Compare
Fix pytest-dev#11122.
d9290ce
to
b41acae
Compare
Rebased, should be good to merge. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wonder if this should be considered an improvement? Feels more like an internal change that does not affect users, so perhaps trivial
would be more appropriate?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It has some behavioral changes in exception handling, however mostly I want plugin authors to see it, so let's keep it as improvement.
Fix #11122.