Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix PEP8 test failures on master #2443

Closed
mdboom opened this issue Sep 20, 2013 · 4 comments
Closed

Fix PEP8 test failures on master #2443

mdboom opened this issue Sep 20, 2013 · 4 comments
Milestone

Comments

@mdboom
Copy link
Member

mdboom commented Sep 20, 2013

When #2099 was merged into master, it has noticed a bunch of PEP8 non-conformance issues in master. As I'd like to focus on getting 1.3.1 ready and not deal with the noise this causes as I merge things back into master, I have disabled this test temporarily. This bug is a reminder to re-enable the test and fix the PEP8 problems properly in the future.

@pelson
Copy link
Member

pelson commented Sep 20, 2013

Reasonable enough. Thanks for the reminder.

@NelleV
Copy link
Member

NelleV commented Sep 20, 2013

Can you specify which file is concerned by this ticket or link to the travis traceback?
I might fix them if I have time this week.

@mdboom
Copy link
Member Author

mdboom commented Sep 20, 2013

The most recent master build has them:

https://travis-ci.org/matplotlib/matplotlib/builds/11588860

Thanks.

pelson added a commit to pelson/matplotlib that referenced this issue Oct 3, 2013
pelson added a commit to pelson/matplotlib that referenced this issue Oct 3, 2013
pelson added a commit to pelson/matplotlib that referenced this issue Oct 3, 2013
NelleV added a commit that referenced this issue Oct 9, 2013
@tacaswell
Copy link
Member

This looks like it should have been closed a while ago, pep8 runs on master again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants