/matplotlib

TST: Always use / in URLs for visual results. #6029

Merged
merged 1 commit into from Feb 19, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
Testing
Projects
None yet
Milestone
  2.1 (next point r...
3 participants
@QuLogic @mdboom @tacaswell
@QuLogic
Member

QuLogic commented Feb 18, 2016

I noticed this on the AppVeyor builds of the image infrastructure PR; the HTML file didn't work because the URLs were incorrect.
@QuLogic QuLogic TST: Always use / in URLs for visual results. 
Backslashes are not valid separators in URLs and maybe only work in IE.

All checks have passed

2 successful checks
continuous-integration/appveyor/pr — AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr — The Travis CI build passed
Details
3e55060

QuLogic added the Testing label Feb 18, 2016

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

QuLogic added the needs_review label Feb 18, 2016

@mdboom
Owner

mdboom commented Feb 18, 2016

👍 from me, once tests pass.

@tacaswell tacaswell added a commit that referenced this pull request Feb 19, 2016

@tacaswell tacaswell Merge pull request #6029 from QuLogic/tests-valid-url 
TST: Always use / in URLs for visual results.
 c0728d2

@tacaswell tacaswell merged commit c0728d2 into matplotlib:master Feb 19, 2016
2 checks passed

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

tacaswell removed the needs_review label Feb 19, 2016

QuLogic deleted the QuLogic:tests-valid-url branch Feb 19, 2016

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