/matplotlib

TST: skip fc-list related tests if not installed #8213

Merged
merged 1 commit into from Mar 8, 2017
+9 −2

Conversation

Reviewers

@afvincent afvincent

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
  2.1 (next point r...
3 participants
@tacaswell @afvincent @QuLogic
@tacaswell
Owner

tacaswell commented Mar 7, 2017

This should gracefully skip the fc-list test if it is not installed on linux or mac.

attn @story645 @matthew-brett
@tacaswell tacaswell TST: skip fc-list related tests if not installed

All checks have passed

5 successful checks
codecov/patch — 100% of diff hit (target 80%)
Details
codecov/project/library — 57.91% (+<.01%) compared to 5faff6e
Details
codecov/project/tests — 98.56% (target 97.9%)
Details
continuous-integration/appveyor/pr — AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr — The Travis CI build passed
Details
c4d5523

tacaswell added this to the 2.1 (next point release) milestone Mar 7, 2017

afvincent referenced this pull request Mar 7, 2017

Merged

Early check for dot binary (from graphviz) when building the doc (fixes #8207) #8221

@afvincent
Contributor

afvincent commented Mar 7, 2017

Just for the record: I guess that this PR purpose is to fix #8152, isn't it?
@afvincent

afvincent approved these changes Mar 7, 2017
View changes

Looks fine to me.

@QuLogic QuLogic merged commit 9a7d9db into matplotlib:master Mar 8, 2017
5 checks passed

5 checks passed

codecov/patch 100% of diff hit (target 80%)
Details
codecov/project/library 57.91% (+<.01%) compared to 5faff6e
Details
codecov/project/tests 98.56% (target 97.9%)
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment