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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
(馃巵) canwecolor馃殌 #14051
(馃巵) canwecolor馃殌 #14051
Conversation
This comment has been minimized.
This comment has been minimized.
59d85c4
to
7d5aa6f
Compare
This comment has been minimized.
This comment has been minimized.
@ilevkivskyi Do you think this is a good one? |
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.
LGTM, just have one suggestion.
pytest.ini
Outdated
@@ -21,7 +21,7 @@ python_functions = | |||
# always run in parallel (requires pytest-xdist, see test-requirements.txt) | |||
# and enable strict mode: require all markers | |||
# to be defined and raise on invalid config values | |||
addopts = -nauto --strict-markers --strict-config | |||
addopts = -nauto --strict-markers --strict-config --color=yes |
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.
Would it be possible to instead add this flag in workflow config? Changing this file will affect local testing as well.
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.
Okay, I've added PYTEST_ADDOPTS
to the workflow file and verified that it cooperates with the addopts
config in pytest.ini
.
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.
ergh, that isn't working.....
Edit: Had to update tox passenv
7d5aa6f
to
29372c1
Compare
This comment has been minimized.
This comment has been minimized.
29372c1
to
4c7f125
Compare
This comment has been minimized.
This comment has been minimized.
4c7f125
to
96f175b
Compare
This comment has been minimized.
This comment has been minimized.
96f175b
to
d7ccbc3
Compare
According to mypy_primer, this change has no effect on the checked open source code. 馃馃帀 |
Last time i checked it wasn't the 1960's. So I think the CI could be colorized.
Configured pytest, tox, mypy(#7771) and pip鹿 (I already already colorized black and isort when I initially added them)
1: Pip doesn't work yet pypa/pip#10909, so this is just a placedholder for when it (hopefully) soon will.