Appveyor: use newer conda packages and only run all tests on one platform #6774

Merged
merged 4 commits into from Jul 18, 2016

Conversation

Projects
None yet
5 participants
Contributor

janschulz commented Jul 17, 2016 edited

Since setting the miktex version as a workaround for the miktex related build failures, the miktex package was updated and an inkscape package was added in conda-forge.

Also install these packages only on one build so that the rest of the builds are shorter...

Also includes the enablement of the conda packages again.

Closes: #6682

@janschulz janschulz Appveyor: use newer conda packages and only run all tests on one plat…
…form
4104449

mdboom added the needs_review label Jul 17, 2016

Contributor

janschulz commented Jul 17, 2016

So, this passes on appveyor and now runs the latex (and other) tests in the Py27/32bit build. This is the first build and takes 30min vs the rest which take 7-8 mins. I'm wondering if putting the full tests in the second run would make more sense so that the first run is fast and one can iterate based on the results of that run (assuming that in most cases changes will only touch non-latex codapaths).

Comments?

Owner

tacaswell commented Jul 17, 2016

Most of the time when I am waiting on appveyor it is due to there being many jobs stacked up, but re-ordering them to put a fast one first sounds like a good idea.

janschulz added some commits Jul 17, 2016

@janschulz janschulz Appveyor: run fast builds first 4032499
@janschulz janschulz Revert "Appveyor: disable building conda packages as they fail the bu…
…ild"

This reverts commit e7c9b56.
c47edde
@janschulz janschulz Appveyor: remove py27 workaround, conda build works
9e0f4af
Contributor

janschulz commented Jul 18, 2016

And it works... Please have a look and merge if satisfied...

@jenshnielsen jenshnielsen merged commit e3fd1b1 into matplotlib:master Jul 18, 2016

3 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 70.369%
Details

QuLogic added this to the 2.1 (next point release) milestone Jul 18, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment