-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
[CI] Add pytest-azurepipelines in pandas-dev #26620
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
Codecov Report
@@ Coverage Diff @@
## master #26620 +/- ##
==========================================
- Coverage 91.85% 91.85% -0.01%
==========================================
Files 174 174
Lines 50722 50722
==========================================
- Hits 46593 46589 -4
- Misses 4129 4133 +4
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #26620 +/- ##
==========================================
- Coverage 91.7% 91.69% -0.01%
==========================================
Files 179 179
Lines 50767 50767
==========================================
- Hits 46555 46551 -4
- Misses 4212 4216 +4
Continue to review full report at Codecov.
|
you can try adding this to setup_env.sh. I think this doesn't work this way because the commandas are executed in different shells (so another option is to do it in the same context
|
@jreback Do you have anything else to implement? |
ci/deps/azure-35-compat.yaml
Outdated
@@ -28,3 +28,4 @@ dependencies: | |||
# for python 3.5, pytest>=4.0.2 is not available in conda | |||
- pytest==4.5.0 | |||
- html5lib==1.0b2 | |||
- pytest-azurepipelines |
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.
Can this be installed from conda-forge?
@TomAugspurger Implemented. Anything else? |
The pytest-azurepiplines docs state that it
Will this break our coverage output for codecov? |
I guess no break is done. Codecov report was also generated in this PR. Is it sufficient? Would you tell me any other way to prove? |
thanks @makbigc cc @pandas-dev/pandas-core we now have a warnings tab in summary: https://dev.azure.com/pandas-dev/pandas/_build/results?buildId=13516&view=results |
Is it better to put
pip install pytest-azurepipelines
insidesetup_env.sh
?