/matplotlib

Tweek tol for test_hist_steplog to fix tests on appveyor #7063

Merged
merged 1 commit into from Sep 8, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
Testing
Projects
None yet
Milestone
  v2.0.0
3 participants
@jenshnielsen @QuLogic @tacaswell
@jenshnielsen
Owner

jenshnielsen commented Sep 8, 2016

No description provided.

jenshnielsen added the needs_review label Sep 8, 2016

jenshnielsen changed the base branch to matplotlib:master from matplotlib:v2.x Sep 8, 2016

@jenshnielsen
Owner

jenshnielsen commented Sep 8, 2016

This is against master to confirm that it fixes appveyor but should probably be backported to 2.x

jenshnielsen added this to the 2.0 (style change major release) milestone Sep 8, 2016

jenshnielsen closed this Sep 8, 2016

jenshnielsen reopened this Sep 8, 2016

@jenshnielsen jenshnielsen added needs_review and removed needs_review labels Sep 8, 2016

@jenshnielsen jenshnielsen Tweek tol for test_hist_steplog to fix tests on appveyor

All checks have passed

3 successful checks
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.362%
Details
725ccd5

jenshnielsen referenced this pull request Sep 8, 2016

Merged

FIX: testings.nose was not installed #7055

@jenshnielsen
Owner

jenshnielsen commented Sep 8, 2016

Ok this finally passed on appveyor. I suggest merging asap to unbreak appveyor on master

@tacaswell tacaswell merged commit 2d9c534 into matplotlib:master Sep 8, 2016
3 checks passed

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.362%
Details

tacaswell removed the needs_review label Sep 8, 2016

jenshnielsen deleted the jenshnielsen:fixhiststeplog_appveyor branch Sep 8, 2016

@QuLogic QuLogic modified the milestone: v1.5.x, 2.0 (style change major release) Sep 9, 2016

@QuLogic
Member

QuLogic commented Sep 9, 2016

Seems like this needs a backport still.
@tacaswell
Owner

tacaswell commented Sep 9, 2016

thanks for the reminder!

QuLogic added the Testing label Sep 9, 2016

@tacaswell tacaswell added a commit that referenced this pull request Sep 9, 2016

@tacaswell tacaswell Merge pull request #7063 from jenshnielsen/fixhiststeplog_appveyor 
TST: Tweek tol for test_hist_steplog to fix tests on appveyor
 6103dd6
@tacaswell
Owner

tacaswell commented Sep 9, 2016

backported to v2.x as 6103dd6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment