Skip to content

Conversation

blueyed
Copy link
Contributor

@blueyed blueyed commented Feb 11, 2019

This is meant to help with AppVeyor builds being being too much.

[skip travis]

@blueyed
Copy link
Contributor Author

blueyed commented Feb 11, 2019

Should we keep/use py27/py37 without xdist only?

@codecov
Copy link

codecov bot commented Feb 12, 2019

Codecov Report

Merging #4767 into features will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##           features    #4767      +/-   ##
============================================
- Coverage     95.69%   95.68%   -0.02%     
============================================
  Files           113      113              
  Lines         25028    25028              
  Branches       2484     2484              
============================================
- Hits          23950    23947       -3     
- Misses          760      762       +2     
- Partials        318      319       +1
Flag Coverage Δ
#docs 29.62% <ø> (+0.07%) ⬆️
#doctesting 29.62% <ø> (+0.07%) ⬆️
#linting 29.62% <ø> (+0.07%) ⬆️
#linux 95.52% <ø> (ø) ⬆️
#nobyte 92.31% <ø> (ø) ⬆️
#numpy 93.1% <ø> (-0.02%) ⬇️
#pexpect 41.99% <ø> (-0.03%) ⬇️
#py27 93.69% <ø> (ø) ⬆️
#py34 91.99% <ø> (+0.26%) ⬆️
#py35 91.99% <ø> (+0.26%) ⬆️
#py36 92.01% <ø> (+0.26%) ⬆️
#py37 93.81% <ø> (-0.02%) ⬇️
#trial 93.1% <ø> (-0.02%) ⬇️
#windows 93.77% <ø> (-0.11%) ⬇️
#xdist 93.77% <ø> (+0.04%) ⬆️
Impacted Files Coverage Δ
src/_pytest/pytester.py 87.03% <0%> (-0.43%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 31c9481...747a8ae. Read the comment docs.

@nicoddemus
Copy link
Member

It seems this saved us some good 20 mins, great job!

On a related note, I have been meaning to try Azure Pipelines at least for the Windows jobs.

@nicoddemus nicoddemus merged commit f63fbf8 into pytest-dev:features Feb 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants