-
-
Notifications
You must be signed in to change notification settings - Fork 2.8k
fix: support TerminalReporter.isatty being called #13462
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.
Outside impact is unclear as others may be using the api as is
We may need to introduce a bool wrapper that support direct checks and calling to obtain the underlying boolean
3cf70bc
to
f87e547
Compare
18b5524
to
8a0687a
Compare
src/_pytest/terminal.py
Outdated
def __init__(self, value: bool): | ||
self._value = value | ||
|
||
def __bool__(self): |
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.
Do we Want to plan for a evenual deprecation of the non call variant?
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.
you mean by issuing a warning when it's used?
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.
I don't think the churn to plugin authors is worth it, given the support code is small.
But we definitely only want to advertise/document the method variant, of course.
9b1085c
to
3af5c0a
Compare
3af5c0a
to
d3ddfec
Compare
Backport to 8.3.x: 💔 cherry-picking failed — conflicts found❌ Failed to cleanly apply cba5c4a on top of patchback/backports/8.3.x/cba5c4a98f65fdcfa484749bc06bcde1fb0865bc/pr-13462 Backporting merged PR #13462 into main
🤖 @patchback |
Backport to 8.4.x: 💚 backport PR created✅ Backport PR branch: Backported as #13483 🤖 @patchback |
Fixes #13461. (cherry picked from commit cba5c4a) Co-authored-by: Martin Fischer <[email protected]>
Fixes #13461.