Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

PEP8 fixes on patches #1399

Merged
merged 1 commit into from Nov 11, 2012

Conversation

Projects
None yet
3 participants
Contributor

NelleV commented Oct 14, 2012

PEP8 fixes on module patches.

Thanks,
N

Member

dmcdougall commented Nov 7, 2012

+1

@dmcdougall dmcdougall commented on the diff Nov 11, 2012

lib/matplotlib/patches.py
@@ -1600,6 +1599,7 @@ def _pprint_styles(_styles, leadingspace=2):
(stylename : styleclass), return a formatted string listing all the
styles. Used to update the documentation.
"""
+ # FIXME pad isn't used, which means leadingspace is not use.
@dmcdougall

dmcdougall Nov 11, 2012

Member

I'll open a separate issue for this once this is merged. Thanks.

@pelson

pelson Nov 13, 2012

Member

@dmcdougall was this ever created?

@dmcdougall

dmcdougall Nov 13, 2012

Member

Yes sir. See #1480.

@dmcdougall dmcdougall added a commit that referenced this pull request Nov 11, 2012

@dmcdougall dmcdougall Merge pull request #1399 from NelleV/pep8_patches
PEP8 fixes on patches
222a358

@dmcdougall dmcdougall merged commit 222a358 into matplotlib:master Nov 11, 2012

1 check passed

default The Travis build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment