/matplotlib

Add OSX to travis tests #6568

Merged
merged 12 commits into from Jun 17, 2016
+23 −20

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
  2.1 (next point r...
5 participants
@jenshnielsen @janschulz @mdboom @tacaswell @QuLogic
@jenshnielsen
Owner

jenshnielsen commented Jun 10, 2016

Installed dependencies from homebrew. Passes on my branch but is unfortunately rather slow ~ 30 min

jenshnielsen added some commits Jun 10, 2016

@jenshnielsen jenshnielsen Remove travis after all trigger 69b5f54
@jenshnielsen jenshnielsen Try running tests on OSX 6e82772
@jenshnielsen jenshnielsen Correct case c2fbf58
@jenshnielsen jenshnielsen Correct bash logic : b6523ca
@jenshnielsen jenshnielsen Remember to install mock on osx 2.7 too 4065053
@jenshnielsen jenshnielsen install inkscape too d053442
@jenshnielsen jenshnielsen Build in verbose mode abc617f
@jenshnielsen jenshnielsen Add more dependencies 0d3aad4
@jenshnielsen jenshnielsen Non verbose

Some checks were not successful

1 failing and 2 successful checks
continuous-integration/appveyor/pr — AppVeyor build failed
Details
continuous-integration/travis-ci/pr — The Travis CI build passed
Details
coverage/coveralls — Coverage increased (+0.002%) to 69.608%
Details
2daee94

mdboom added the needs_review label Jun 10, 2016

tacaswell added this to the 2.0.1 (next bug fix release) milestone Jun 11, 2016

jenshnielsen added some commits Jun 16, 2016

@jenshnielsen jenshnielsen Don't install ghostscript and inkscape on osx

All checks have failed

2 errored checks
continuous-integration/appveyor/pr — AppVeyor build cancelled
Details
continuous-integration/travis-ci/pr — The Travis CI build could not complete due to an error
Details
d7ca847
@jenshnielsen jenshnielsen Use ccache on OSX

Some checks were not successful

1 failing and 2 successful checks
continuous-integration/appveyor/pr — AppVeyor build failed
Details
continuous-integration/travis-ci/pr — The Travis CI build passed
Details
coverage/coveralls — Coverage increased (+0.6%) to 70.245%
Details
7a25978
@jenshnielsen
Owner

jenshnielsen commented Jun 16, 2016

I I think this is ready to merge. I have added ccache support to OSX and removed the pdf and svg tests from OSX to save time.
@jenshnielsen
Owner

jenshnielsen commented Jun 16, 2016

@janschulz Not sure what has happened to appveyor here perhaps related to conda updates?
@jenshnielsen jenshnielsen OSX boxes are only 1 core

Some checks were not successful

1 failing and 2 successful checks
continuous-integration/appveyor/pr — AppVeyor build failed
Details
continuous-integration/travis-ci/pr — The Travis CI build passed
Details
coverage/coveralls — Coverage increased (+0.6%) to 70.245%
Details
5bc35b1
@janschulz
Contributor

janschulz commented Jun 16, 2016 • edited

This seems to be conda update...? I added conda/conda#2732

jenshnielsen closed this Jun 17, 2016

jenshnielsen removed the needs_review label Jun 17, 2016

jenshnielsen reopened this Jun 17, 2016

jenshnielsen added the needs_review label Jun 17, 2016

@tacaswell tacaswell merged commit 21e16c3 into matplotlib:master Jun 17, 2016
2 of 3 checks passed

2 of 3 checks passed

continuous-integration/appveyor/pr AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.6%) to 70.245%
Details

tacaswell removed the needs_review label Jun 17, 2016

jenshnielsen deleted the jenshnielsen:travisosx branch Jun 17, 2016

@QuLogic QuLogic modified the milestone: 2.1 (next point release), 2.0.1 (next bug fix release) Jul 18, 2016

anntzer referenced this pull request Jan 23, 2017

Merged

Stop codecov from posting messages #7926

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