/AzurePipelines run pytest-CI

Codecov Report

Merging #4831 into master will increase coverage by 0.19%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4831      +/-   ##
==========================================
+ Coverage   95.48%   95.68%   +0.19%     
==========================================
  Files         113      113              
  Lines       25163    25163              
  Branches     2498     2498              
==========================================
+ Hits        24026    24076      +50     
+ Misses        803      771      -32     
+ Partials      334      316      -18

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 7428064...d5d1903. Read the comment docs.

Hmmm realized that we need to skip coverage for pypy because it is way too slow. It has been running now for 30m, and only reached 60% of the tests. 😓

Ready for review.

Hmm bummer, build time jumped from 15m to 30m because of coverage. 😞

But of course, much faster than AppVeyor. 👍

@blueyed I'm not sure the flags adjustment to codecov helped, the codecov/project check is still timing out for me. 😞

@nicoddemus
too bad, sorry.
It is likely more/also related to the number of uploads (which we now have increased with azure).

The commit itself can be seen on https://codecov.io/gh/pytest-dev/pytest/tree/5494eb1948351d44ff58288dcfc500d1a584aed6 - but is not having the "Changes".

Those are due to skipping Linux/MacOS/Travis though likely, no?

I'll go ahead and amend this.

Those are due to skipping Linux/MacOS/Travis though likely, no?

Right, most likely.

I'll go ahead and amend this.

OK, thanks!

@nicoddemus
I think you want to drop AppVeyor completely soon now, but we should remove coverage reporting from it at least as soon as possible.

Yeah not sure how much py37-freeze is a blocker on Azure anyway. I will give it a shot later, and if I can't get it to work I think it is fine to just assume Azure officially and drop AppVeyor.