/matplotlib

BUG: fix ListedColormap._resample, hence plt.get_cmap; closes #6025 #6036

Merged
merged 1 commit into from Feb 21, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
  v1.5.2
3 participants
@efiring @QuLogic @tacaswell
@efiring
Owner

efiring commented Feb 20, 2016

No description provided.

efiring added the needs_review label Feb 20, 2016

efiring added this to the 1.5.2 (Critical bug fix release) milestone Feb 20, 2016

efiring closed this Feb 20, 2016

efiring removed the needs_review label Feb 20, 2016

@efiring
Owner

efiring commented Feb 20, 2016

Closing and reopening in an attempt to wake up Travis.

efiring reopened this Feb 20, 2016

efiring added the needs_review label Feb 20, 2016

@QuLogic
Member

QuLogic commented Feb 20, 2016

PEP8 errors:

AssertionError: 1 != 0 : Found code syntax errors (and warnings):
/home/travis/build/matplotlib/matplotlib/lib/matplotlib/tests/test_colors.py:20:1: E302 expected 2 blank lines, found 1
@efiring
Owner

efiring commented Feb 20, 2016

Sometimes I could throttle PEP8...
@efiring efiring BUG: fix ListedColormap._resample, hence plt.get_cmap; closes #6025

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
8d2da2c

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

@tacaswell tacaswell Merge pull request #6036 from efiring/ListedColormap_resample 
BUG: fix ListedColormap._resample, hence plt.get_cmap; closes #6025
 0a72724

@tacaswell tacaswell merged commit 0a72724 into matplotlib:master Feb 21, 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 21, 2016

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

@tacaswell tacaswell Merge pull request #6036 from efiring/ListedColormap_resample 
BUG: fix ListedColormap._resample, hence plt.get_cmap; closes #6025
 9dca4d8
@tacaswell
Owner

tacaswell commented Feb 21, 2016

backported to 1.5.x as 9dca4d8

@tacaswell tacaswell added a commit to tacaswell/matplotlib that referenced this pull request May 22, 2016

@tacaswell tacaswell Merge pull request #6036 from efiring/ListedColormap_resample 
BUG: fix ListedColormap._resample, hence plt.get_cmap; closes #6025
 152fa08

efiring deleted the efiring:ListedColormap_resample branch Jun 23, 2016

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