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

pep8ify examples (part2) #3488

Closed
wants to merge 7 commits into from
Closed

pep8ify examples (part2) #3488

wants to merge 7 commits into from

Conversation

twmr
Copy link
Contributor

@twmr twmr commented Sep 9, 2014

Continuation of #3425

This time I converted

color
event_handling
images_contours_and_fields
lines_bars_and_marker
misc
mplot3d
pie_and_polar_charts

TODO

  • remove newline(s) after class declaration

@tacaswell tacaswell added this to the v1.5.x milestone Sep 9, 2014
('Mult. masters', 8),
('Glyph names', 9),
('External stream', 10)
)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you want me to convert these lists of tuples to lists of strings? I could use enumerate in the client code to get the indices.

@twmr twmr force-pushed the pep8p2 branch 2 times, most recently from c2839ad to de8b5c4 Compare September 9, 2014 15:52
Signed-off-by: Thomas Hisch <t.hisch@gmail.com>
Signed-off-by: Thomas Hisch <t.hisch@gmail.com>
Signed-off-by: Thomas Hisch <t.hisch@gmail.com>
Signed-off-by: Thomas Hisch <t.hisch@gmail.com>
Signed-off-by: Thomas Hisch <t.hisch@gmail.com>
Signed-off-by: Thomas Hisch <t.hisch@gmail.com>
@tacaswell
Copy link
Member

@thisch This needs to be re-based.

@twmr
Copy link
Contributor Author

twmr commented Oct 18, 2014

This PR is obsolete and can be closed.

@twmr twmr closed this Oct 18, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants