-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
terminal: use "yellow" with any "xpassed" tests #6425
Conversation
@@ -0,0 +1 @@ | |||
Use "yellow" main color with any XPASSED tests. |
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.
Use "yellow" main color with any XPASSED tests. | |
Use yellow main color with any XPASSED tests. |
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.
Thanks!
@@ -0,0 +1 @@ | |||
Use "yellow" main color with any XPASSED tests. |
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.
IIUC there is no actual change here, because the previous code (probably unintentionally) used the fallback color, which is also yellow. So the changelog can be dropped as it might be confusing.
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.
..only if there is no passed one.
So now it is yellow always with an xpass.
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.
Oh right, I missed that this talks about the main color.
(cherry picked from commit 13baab7) Ref: pytest-dev#6425 Ref: pytest-dev#449.
Closes #449.